Bug 239056

Summary: When you grab the map, the mouse cursor hand does not grab
Product: [Applications] marble Reporter: Richard Hartmann <richih-kde>
Component: generalAssignee: marble-bugs
Status: RESOLVED FIXED    
Severity: normal CC: nienhueser
Priority: NOR    
Version First Reported In: unspecified   
Target Milestone: 0.10 (KDE 4.5)   
Platform: Debian unstable   
OS: Linux   
Latest Commit: Version Fixed/Implemented In:
Sentry Crash Report:

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