[Marble-devel] Review Request: Zoom via Ctrl+Mouse Selection Rectangle

[Dennis Nienhüser]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/2666/#review3777
-----------------------------------------------------------


Looks nice and clean code-wise, haven't tested it yet though. The new regionSelected() signal in MarbleWidget.h should get some doxygen comment.


- Dennis


On 2010-01-21 15:45:20, Torsten Rahn wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://reviewboard.kde.org/r/2666/
> -----------------------------------------------------------
> 
> (Updated 2010-01-21 15:45:20)
> 
> 
> Review request for marble.
> 
> 
> Summary
> -------
> 
> Fixes [Bug 183512] move viewport / move earth inside window
> 
> 
> This addresses bug 183512.
>     https://bugs.kde.org/show_bug.cgi?id=183512
> 
> 
> Diffs
> -----
> 
>   trunk/KDE/kdeedu/marble/src/QtMainWindow.cpp 1077344 
>   trunk/KDE/kdeedu/marble/src/lib/MarbleWidget.h 1077287 
>   trunk/KDE/kdeedu/marble/src/lib/MarbleWidget.cpp 1077287 
>   trunk/KDE/kdeedu/marble/src/marble_part.cpp 1077287 
> 
> Diff: http://reviewboard.kde.org/r/2666/diff
> 
> 
> Testing
> -------
> 
> Tested with Qt and KDE version of Marble
> 
> 
> Thanks,
> 
> Torsten
> 
>