After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 594617 - Touchpad left-handed bug fixes
Touchpad left-handed bug fixes
Status: RESOLVED FIXED
Product: gnome-settings-daemon
Classification: Core
Component: plugins
2.26.x
Other Linux
: Normal normal
: ---
Assigned To: gnome-settings-daemon-maint
gnome-settings-daemon-maint
Depends on:
Blocks:
 
 
Reported: 2009-09-09 10:59 UTC by Bastien Nocera
Modified: 2009-09-09 11:49 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
patch (2.16 KB, patch)
2009-09-09 11:00 UTC, Bastien Nocera
reviewed Details | Review

Description Bastien Nocera 2009-09-09 10:59:14 UTC
From https://bugzilla.redhat.com/show_bug.cgi?id=483639

1) Don't swap mouse buttons for left-handers on the touchpad (otherwise a tap would be a right-click)
2) If XInput is supported, don't switch the core pointer to left-handed, as it would cancel our other settings

This is the patch for 2.26, if you deem it necessary to have it:
http://cvs.fedoraproject.org/viewvc/rpms/gnome-settings-daemon/F-11/gnome-settings-daemon-2.26.1-fix-touchpad.patch?view=markup
Comment 1 Bastien Nocera 2009-09-09 11:00:22 UTC
Created attachment 142772 [details] [review]
patch
Comment 2 Jens Granseuer 2009-09-09 11:19:00 UTC
Comment on attachment 142772 [details] [review]
patch

Could you please elaborate on 2)? What's the effect of not swapping the core pointer with XInput, and why do we not want to do that? I think it'd be helpful to spell that out in the comments.

Patch looks good.
Comment 3 Bastien Nocera 2009-09-09 11:38:12 UTC
(In reply to comment #2)
> (From update of attachment 142772 [details] [review])
> Could you please elaborate on 2)? What's the effect of not swapping the core
> pointer with XInput,

The core pointer is the aggregation of all the devices plugged into your system. If you change the configuration of each device through XInput, then change the core pointer, it's as if you didn't do anything.

If you don't have XInput support, then the only "device" you have access to is the core pointer.

I think it's pretty self-explanatory if you understand what the CorePointer is...

> and why do we not want to do that? I think it'd be helpful
> to spell that out in the comments.
Comment 4 Bastien Nocera 2009-09-09 11:49:04 UTC
Committed with slightly better comments. Note that Peter did check this patch, and vouched for it as well.

commit 6a3bedfcb9be30b883a145d7e4ce83fd9cbc3e25
Author: Bastien Nocera <hadess@hadess.net>
Date:   Wed Sep 9 11:59:21 2009 +0100

    Bug 594617 - Touchpad left-handed bug fixes
    
    1) Don't swap mouse buttons for left-handers on the touchpad (otherwise
    a tap would be a right-click)
    2) If XInput is supported, don't switch the core pointer to left-handed,
    as it would cancel our other settings