Bug 239056 - When you grab the map, the mouse cursor hand does not grab
Summary: When you grab the map, the mouse cursor hand does not grab
Status: RESOLVED FIXED
Alias: None
Product: marble
Classification: Applications
Component: general (show other bugs)
Version: unspecified
Platform: Debian unstable Linux
: NOR normal
Target Milestone: 0.10 (KDE 4.5)
Assignee: marble-bugs
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2010-05-27 22:23 UTC by Richard Hartmann
Modified: 2010-06-03 00:00 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Richard Hartmann 2010-05-27 22:23:10 UTC
Version:           unspecified (using KDE 4.4.3) 
OS:                Linux

When you grab the map, you need to move the mouse a tiny little bit before the mouse cursor becomes a closed hand.

Reproducible: Always

Steps to Reproduce:
1) Start marble
2) Grab map
3) Look at mouse cursour
4) Wiggle the mouse a bit
5) Look at the mouse cursor again

Actual Results:  
Open hand

Expected Results:  
Closed hand

OS: Linux (i686) release 2.6.32-4-686
Compiler: cc
Comment 1 Dennis Nienhüser 2010-05-30 16:38:22 UTC
SVN commit 1132367 by nienhueser:

Don't leave the method early when no drag operation happens: The cursor shape still needs to be adjusted.
Needs regression testing and a backport to 4.5 and 4.4 afterwards.
CCBUG: 239056

 M  +3 -4      MarbleWidgetInputHandler.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1132367
Comment 2 Dennis Nienhüser 2010-06-02 22:21:50 UTC
SVN commit 1133911 by nienhueser:

Don't leave the method early when no drag operation happens: The cursor shape still needs to be adjusted.
Needs regression testing and a backport to 4.5 and 4.4 afterwards.
CCBUG: 239056
Fix minor logic error introduced in rev. 1132367.
Backport of commits 1132367 and 1133908


 M  +3 -4      MarbleWidgetInputHandler.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1133911
Comment 3 Dennis Nienhüser 2010-06-02 22:25:47 UTC
Fixed in trunk and 4.5, thanks for the bug report.
Comment 4 Richard Hartmann 2010-06-02 22:56:29 UTC
There will be a 4.4.5 release, so porting to 4.4 makes sense, as well.

Thanks for the fix :)
Comment 5 Dennis Nienhüser 2010-06-03 00:00:20 UTC
SVN commit 1133938 by nienhueser:

Don't leave the method early when no drag operation happens: The cursor shape still needs to be adjusted.
Needs regression testing and a backport to 4.5 and 4.4 afterwards.
BUG: 239056
Fix minor logic error introduced in rev. 1132367.
Backport of commits 1132367 and 1133908



 M  +4 -4      MarbleWidgetInputHandler.cpp  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1133938