Bug 24577 - Mouse Jumping Down when clicking
Summary: Mouse Jumping Down when clicking
Status: RESOLVED FIXED
Alias: None
Product: xorg
Classification: Unclassified
Component: Server/DDX/dmx (show other bugs)
Version: git
Hardware: x86 (IA32) Linux (All)
: medium normal
Assignee: dmx-bugs
QA Contact: Xorg Project Team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-10-16 12:41 UTC by Lee Leahu
Modified: 2009-10-21 07:41 UTC (History)
1 user (show)

See Also:
i915 platform:
i915 features:


Attachments
Correct the calls to InitValuatorAxisStruct - use loop counters (2.91 KB, patch)
2009-10-16 23:01 UTC, Lee Leahu
no flags Details | Splinter Review
dmx - fix device valuator axis information (3.06 KB, patch)
2009-10-17 21:35 UTC, Lee Leahu
no flags Details | Splinter Review
patch #30512 regenerated using git-format-patch (3.06 KB, patch)
2009-10-18 18:32 UTC, Lee Leahu
no flags Details | Splinter Review
correct array references in previous patch (3.16 KB, patch)
2009-10-19 14:51 UTC, Lee Leahu
no flags Details | Splinter Review

Description Lee Leahu 2009-10-16 12:41:56 UTC
xorg-server - GIT Commit - 25344ba7f7845654364d62bf15322b3b79465bd9

When openbox or KDE window managers are running, any left clicks performed in side of a application window results in the mouse pointer jumping downwards.

Xdmx doesn't appear to recognize the mouse jumps.  Some debugging was performed using openbox and there's a call to XAllowEvents to replay the mouse position.  If that call is disabled, the mouse doesn't jump (but clicks are not functional after that).  

If you hold the mouse still and just left click once, you'll notice the mouse pointer jumps down.   Then immediatley move the mouse just a tad from there and you'll notice that mouse pointer jumps back up and acts as if it had never jumped down.
Comment 1 Lee Leahu 2009-10-16 18:05:58 UTC
Here's a simple test:

With only openbox and xterm running, and without moving the mouse at all, click and release the left button.

With DMX_BACKEND_DEBUG and DMX_EVENTS_DEBUG, I get this:

For the button press event:

(..) dmx: dmxEnqueue: Enqueuing type=4 detail=0×1
(..) dmx: dmxBackendProcessInput: myScreen=1 relative=0 last=657,514 center=640,512
(..) dmx: dmxBackendProcessInput: myScreen=1 relative=0 last=657,514 center=640,512
(..) dmx: dmxBackendCollectEvents: MotionNotify 1/1 (mi 1) newscreen=0: 574 390 (e=0; last=657,514)
(..) dmx: Absolute move: 1854,390 (r=1280×1024+0+0 s=1280×1024)


For the button release event: 

(..) dmx: dmxEnqueue: Enqueuing type=5 detail=0×1
(..) dmx: dmxBackendCollectEvents: MotionNotify 1/1 (mi 1) newscreen=0: 574 488 (e=0; last=657,514)
(..) dmx: Absolute move: 1854,488 (r=1280×1024+0+0 s=1280×1024)
(..) dmx: dmxCoreMotion(1854,488,0) dmxGlobalX=1854 dmxGlobalY=390
(..) dmx: dmxBackendUpdatePosition(1854,488) my=1 mi=1 rel=0 topscreen=1 same=1 offscreen=0 offthis=0
(..) dmx: dmxBackendProcessInput: myScreen=1 relative=0 last=657,514 center=640,512

Notice the Y value of newscreen.

Comment 2 Lee Leahu 2009-10-16 23:01:12 UTC
Created attachment 30494 [details] [review]
Correct the calls to InitValuatorAxisStruct - use loop counters
Comment 3 Lee Leahu 2009-10-17 21:35:14 UTC
Created attachment 30512 [details] [review]
dmx - fix device valuator axis information
Comment 4 Lee Leahu 2009-10-18 18:32:48 UTC
Created attachment 30546 [details] [review]
patch #30512 regenerated using git-format-patch
Comment 5 Lee Leahu 2009-10-19 14:51:49 UTC
Created attachment 30573 [details] [review]
correct array references in previous patch
Comment 6 Lee Leahu 2009-10-21 07:41:27 UTC
Closing.

Patch commited to git master as 664a8e37fd83141974b772980f680b94e48b4f87.


Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.