Unity Control Center

Merge lp://staging/~laney/unity-control-center/lp1132063 into lp://staging/unity-control-center

  1. lp1132063
  2. Merge into trunk
Proposed by Iain Lane
Status: Work in progress
Proposed branch: lp://staging/~laney/unity-control-center/lp1132063
Merge into: lp://staging/unity-control-center
Diff against target: 94 lines (+44/-3)
1 file modified
panels/mouse/gsd-input-helper.c (+44/-3)
To merge this branch: bzr merge lp://staging/~laney/unity-control-center/lp1132063
Reviewer Review Type Date Requested Status
Sebastien Bacher Needs Information
PS Jenkins bot (community) continuous-integration Approve
Review via email: mp+237636@code.staging.launchpad.net

Commit message

Use XI2 to detect mice, in addition to the current XI method.

Description of the change

See the linked bug, there's problems detecting mice which means that the settings are sometimes hidden when they shouldn't be.

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :

PASSED: Continuous integration, rev:12792
http://jenkins.qa.ubuntu.com/job/unity-control-center-ci/114/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-control-center-utopic-amd64-ci/16
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-control-center-utopic-armhf-ci/16
    SUCCESS: http://jenkins.qa.ubuntu.com/job/unity-control-center-utopic-i386-ci/16

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/unity-control-center-ci/114/rebuild

review: Approve (continuous-integration)
Reply
Revision history for this message
Sebastien Bacher (seb128) wrote :

thanks, is that something upstream g-c-c is doing? if not should we upstream the change?

Reply
Revision history for this message
Sebastien Bacher (seb128) :
review: Needs Information
Reply
Revision history for this message
Iain Lane (laney) wrote :

On Thu, Oct 09, 2014 at 06:35:12AM -0000, Sebastien Bacher wrote:
> thanks, is that something upstream g-c-c is doing? if not should we upstream the change?

No, and yes probably, I wanted code review here first

--
Iain Lane [ <email address hidden> ]
Debian Developer [ <email address hidden> ]
Ubuntu Developer [ <email address hidden> ]

Reply
Revision history for this message
Sebastien Bacher (seb128) wrote :

The code by itself seems fine to me, I don't know enough xinput to have an opinion on whether that's the right way to check for the device/the most suited api. Not sure we have anyone in the team who does ... upstream might be better reviewer there?

Reply
Revision history for this message
Iain Lane (laney) wrote :

Ok

Reply
Revision history for this message
Iain Lane (laney) wrote :

If anyone has a non-multitouch trackpad, please check this. Mine has that and so I can't see if the mouse detection is buggy.

Reply
Revision history for this message
Sebastien Bacher (seb128) wrote :

iirc you started discussing that upstream, could you add the bugzilla reference here?

Reply
Revision history for this message
Iain Lane (laney) wrote :

https://bugzilla.gnome.org/show_bug.cgi?id=738220

On Fri, Dec 12, 2014 at 02:55:15PM -0000, Sebastien Bacher wrote:
> iirc you started discussing that upstream, could you add the bugzilla reference here?
> --
> https://code.launchpad.net/~laney/unity-control-center/lp1132063/+merge/237636
> You are the owner of lp:~laney/unity-control-center/lp1132063.
>

--
Iain Lane [ <email address hidden> ]
Debian Developer [ <email address hidden> ]
Ubuntu Developer [ <email address hidden> ]

Reply

Unmerged revisions

12792. By Iain Lane

mouse: Use XI2 to detect mice, in addition to the existing method

Sometimes, `xinput' reports output like the following

⎡ Virtual core pointer id=2 [master pointer (3)]
⎜ ↳ Virtual core XTEST pointer id=4 [slave pointer (2)]
⎜ ↳ Logitech USB Receiver id=9 [slave pointer (2)]
⎣ Virtual core keyboard id=3 [master keyboard (2)]
    ↳ Virtual core XTEST keyboard id=5 [slave keyboard (3)]
    ↳ Power Button id=6 [slave keyboard (3)]
    ↳ Power Button id=7 [slave keyboard (3)]
    ↳ Logitech USB Receiver id=8 [slave keyboard (3)]

and g-c-c doesn't detect a mouse, which hides the settings.

We try to check for an XI2 XIMasterPointer device without an XITouchClass class
as a way of detcting mice-but-not-touchpads.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches