Bug 106631 - mouse configuration during installation
Summary: mouse configuration during installation
Status: RESOLVED FIXED
: 114625 (view as bug list)
Alias: None
Product: SUSE LINUX 10.0
Classification: openSUSE
Component: Installation (show other bugs)
Version: Beta 2
Hardware: Other All
: P5 - None : Major
Target Milestone: ---
Assignee: Marcus Schaefer
QA Contact: Klaus Kämpf
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2005-08-23 08:52 UTC by Jakub Friedl
Modified: 2005-09-01 07:57 UTC (History)
5 users (show)

See Also:
Found By: Other
Services Priority:
Business Priority:
Blocker: ---
Marketing QA Status: ---
IT Deployment: ---


Attachments
mouse_proposal.dif (1.04 KB, patch)
2005-08-23 13:55 UTC, Marcus Schaefer
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Jakub Friedl 2005-08-23 08:52:36 UTC
Changing the mouse type doesnt influence mouse behaviour during the installation.
Isnt it setting a mouse for the text mode only?
Comment 1 Marcus Schaefer 2005-08-23 09:09:10 UTC
yes changing the mouse type doesn't influence the X11 mouse behaviour 
but the GPM mouse parameters for the console. The module is only capable 
of doing this nothing more 
Comment 2 Jakub Friedl 2005-08-23 10:08:52 UTC
Well, that is the problem. This dialog during the installation should provide a
way to set up the mouse (X) correctly if the autodetection failed.

Even the manuals are writen that way.
Comment 3 Marcus Schaefer 2005-08-23 10:17:53 UTC
if you can provide a new X11 misc extension which is able to apply all 
kind of mice on the running X-Server we can enable it again otherwise 
this doesn't make any sense because a mouse which has not been detected 
by the kernel and libhd cannot be applied by the X11 misc extension. 
Comment 4 Jakub Friedl 2005-08-23 10:36:24 UTC
Then it should be made clear to users. the current documentation says:

"If &yast; failed to detect your mouse automatically, press
    <keycap>Tab</keycap> in the suggestion window several times until
    <guimenu>Mouse</guimenu> is selected. Then use <keycap>Space</keycap> to
    open the dialog in which to set the mouse type."

Even without reading the manual, any user would expect this dialog to set X11
mouse, and would try to use it if her mouse hadn't been autodetected correctly.
She would be very frustrated then, not knowing why her mouse didn't respond to
the settings.

If we are really not able to enable X11 mouse configuration here, then it should
 be made clear and obvious both in the documentation and in the dialog.
Comment 5 Marcus Schaefer 2005-08-23 10:58:06 UTC
- Good heavens that's not a critical bug ;) If you want the docu-team 
  to change something file a bug report there I'm not responsible for that 
 
- unfortunately we have text freeze for 10.0 so I'm not allowed to change 
  the text displayed in the GUI. Adding text should be avoided but is possible 
  changing text will trigger a complete new translation round which is not 
  allowed after freeze 
 
 
Comment 6 Klara Cihlarova 2005-08-23 11:25:10 UTC
Is not it really possible fix it or retrieve old X mouse dialog? Sorry if my 
question sounds silly, I don't know amount of changes which you made in the 
mouse module. 
 
At worst we can add a few words about it to Release Notes to prevent users 
complaints and bug reports of this problem. 
 
Comment 7 Marcus Schaefer 2005-08-23 12:11:58 UTC
what for the world does it help to revert the old X mouse dialog ? There 
is no such dialog and of course the changes cannot be merged. As I already 
said so please listen: 
 
 - the X11 misc extension is able to apply mouse with a maximum of 3 
   buttons (no wheel) talking standard PS/2 or old serial mouse 
   protocol. All of these devices are safely detected automatically 
   by the kernel. A device which is not detected CANNOT be activated 
   with the very old misc extension stuff. It does not make sense to 
   provide a NON working dialog as we did in the past. 
 
The yast mouse dialog is for configuration of GPM and not more. I agree 
we have to clear the texts but it's too late for 10.0 therefore I set this 
to later 
 
If you want to add a paragraph in the release notes or in the documentation 
please refer to the responsible people it's not me to do that :-)  
 
Thanks 
Comment 8 Klara Cihlarova 2005-08-23 12:24:40 UTC
OK. Is it then possible take away it from installation completely?  
Comment 9 Jiří Suchomel 2005-08-23 12:45:09 UTC
Yes, I think this should be easy and correct solution: jsrain?
Comment 10 Jiri Srain 2005-08-23 12:55:59 UTC
Technically, it is of course no problem. It can be remove completly (via the 
control file), or only from X (Qt) installation (mouse_proposal.ycp has to 
return empty map if running under X - similar to yast2-x11). 
Comment 11 Marcus Schaefer 2005-08-23 13:09:12 UTC
Ok, Jiri please remove it from the control file. Thanks 
Comment 12 Jiri Srain 2005-08-23 13:13:42 UTC
I talked about the technical possibilities. But according to comment #7, it 
should stau there for the NCurses configuraiton. So, wouldn't it be better if 
it behaved similar to X11 proposal? 
Comment 13 Marcus Schaefer 2005-08-23 13:28:17 UTC
why should it stay for ncurses configuration ? 
 
- YaST in ncurses mode is not capable of using a mouse 
- SuSE linux does not activate GPM by default even in runlevel 3 
 
For the installation I see no reason to urgently provide that dialog. 
A user who wants to use GPM has to activate it manually. 
 
 
Do you mean a proposal check like this: 
 
if ( 
    (! Arch::x11_setup_needed ()) || 
    (! Installation::x11_setup_needed ()) 
) { 
  ... 
} 
 
? 
Comment 14 Jiri Srain 2005-08-23 13:41:47 UTC
Hmm, and what did it set in ncurses until now? Didn't it set GPM for the 
installed system? 
 
The check shouls IMHO look like 
 
if (Linuxrc::text ()) 
  behave_normally 
else 
  do_not_put_mouse_to_proposal 
Comment 15 Marcus Schaefer 2005-08-23 13:54:22 UTC
It set the GPM parameters for the installed system, yes that's the 
task the module has been made for *originally* :-) and the task it will 
fulfill in the future 
 
Ok, please check the attached patch 
Comment 16 Marcus Schaefer 2005-08-23 13:55:14 UTC
Created attachment 47216 [details]
mouse_proposal.dif
Comment 17 Jiri Srain 2005-08-23 13:58:57 UTC
The patch seems to look fine. 
Comment 18 Marcus Schaefer 2005-08-23 14:12:11 UTC
fine ;) I will include it now. Thanks for helping  
Comment 19 Thomas Roelz 2005-08-23 15:25:36 UTC
So did I get that right?

o NCurses: Mouse module present in proposal. Sets GPM parameters.

o Qt: Mouse module not present in proposal.

In Qt-YaST would it be possible to call the SaX2 mouse module at this point in
the installation and let it do whatever it does? This would be more consistent
but I assume it's impossible because X is not configured yet?

Of course the docu talks nonsense when it says "If your mouse could not
be detected, do this and that etc." This has to be corrected in any case.
Comment 20 Marcus Schaefer 2005-09-01 07:57:22 UTC
*** Bug 114625 has been marked as a duplicate of this bug. ***