Bug 168744 - Double click should zoom in
Summary: Double click should zoom in
Status: RESOLVED FIXED
Alias: None
Product: marble
Classification: Applications
Component: general (show other bugs)
Version: unspecified
Platform: Ubuntu Linux
: NOR wishlist
Target Milestone: ---
Assignee: Torsten Rahn
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2008-08-08 15:32 UTC by Michael Greifeneder
Modified: 2010-01-27 20:59 UTC (History)
2 users (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments
marble-doubleclick-zooms.diff (6.31 KB, patch)
2010-01-09 01:07 UTC, Dennis Nienhüser
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Greifeneder 2008-08-08 15:32:35 UTC
Version:            (using KDE 4.1.0)
Installed from:    Ubuntu Packages

Currently a double click does nothing in Marble. 
It should zoom in one level and center where the click was done.
Comment 1 Torsten Rahn 2008-08-09 13:57:13 UTC
Yes, this is on our TODO.
Comment 2 Dennis Nienhüser 2010-01-09 01:07:52 UTC
Created attachment 39705 [details]
marble-doubleclick-zooms.diff

Please test the attached patch.

Besides zooming in at the mouse position on double clicks, it increases the timeout for the left mouse button popup a bit (from 250 to 400 ms) to give you a chance of double clicking without getting a menu in your way.
The zoom value is 80, that should be equivalent to zooming in twice.
Comment 3 Dennis Nienhüser 2010-01-25 20:56:41 UTC
SVN commit 1080181 by nienhueser:

Add support for changing the observer position via GeoDataLookAt objects (~ camera distance to some ground point). Generalize jump to target animations and use them for user interaction when enabled in settings. Implement double click to zoom in. Fix rubber band getting stuck when released over a float item.
CCBUG: 168744


 M  +15 -2     MarbleControlBox.cpp  
 M  +63 -28    MarbleMap.cpp  
 M  +34 -1     MarbleMap.h  
 M  +5 -2      MarbleMap_p.h  
 M  +153 -40   MarblePhysics.cpp  
 M  +41 -16    MarblePhysics.h  
 M  +185 -197  MarbleWidget.cpp  
 M  +42 -17    MarbleWidget.h  
 M  +132 -77   MarbleWidgetInputHandler.cpp  
 M  +3 -2      MarbleWidgetInputHandler.h  
 M  +14 -0     global.h  


WebSVN link: http://websvn.kde.org/?view=rev&revision=1080181
Comment 4 Dennis Nienhüser 2010-01-27 20:59:57 UTC
Fixed in trunk, will be available with KDE SC 4.5. Too many changes for a backport to 4.4, I'm afraid.