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 710802 - Implement Cursor Movement
Implement Cursor Movement
Status: RESOLVED FIXED
Product: mousetrap
Classification: Other
Component: General
unspecified
Other Linux
: Normal major
: ---
Assigned To: Logan H
mousetrap-maint
Depends on:
Blocks:
 
 
Reported: 2013-10-24 13:09 UTC by Roland
Modified: 2014-09-13 02:10 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Partial fix to bug (2.40 KB, patch)
2013-12-05 04:39 UTC, Logan H
committed Details | Review

Description Roland 2013-10-24 13:09:55 UTC
Outdated and deprecated code to move the cursor needs to be removed and replaced with current GDK pointer functions. Cursor must update properly on screen according to cursor movement selection.
Comment 1 Logan H 2013-12-05 04:39:39 UTC
Created attachment 263559 [details] [review]
Partial fix to bug

This is a partial fix to the bug. Depreciated functions have been removed from move. Depreciated code from click still remains. New implementation is using GDK.Display to get the device manager, pointer, and screen. WARP is being used to move the cursor on the screen. 

Click code still needs to be fixed, before clicking can be implemented. Also, the code can't be fully tested until tracking, joystick, and screen are functioning.
Comment 2 Heidi Ellis 2014-01-31 20:25:10 UTC
(In reply to comment #1)
> Created an attachment (id=263559) [details] [review]
> Partial fix to bug
> 
> This is a partial fix to the bug. Depreciated functions have been removed from
> move. Depreciated code from click still remains. New implementation is using
> GDK.Display to get the device manager, pointer, and screen. WARP is being used
> to move the cursor on the screen. 
> 
> Click code still needs to be fixed, before clicking can be implemented. Also,
> the code can't be fully tested until tracking, joystick, and screen are
> functioning.

Code was committed: https://git.gnome.org/browse/mousetrap/commit/?h=gnome3-wip&id=466472819c6f995ca7005a5bffa82af4d688b2b7

Still need to implement the cursor movement.
Comment 3 Stoney Jackson 2014-01-31 20:26:15 UTC
Review of attachment 263559 [details] [review]:

Committed.