[Marble-devel] Review Request 123097: Added horizontal and vertical resize to GroundOverlay

Torsten Rahn tackat at kde.org
Sun Mar 22 07:43:17 UTC 2015


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/123097/#review77899
-----------------------------------------------------------


Possible next task: Add rotation via dragging: 

Check this tutorial to see how it works in inkscape: http://www.apm-designs.com/inkscape-tutorials/creating-and-editing-a-rectangle-in-inkscape-048  This behavior is kind of a standard for all drawing tools. It would be nice if it worked the same way in Marble for GroundOverlays (yes that might require to change the "node circles" to arrows as seen in the tutorial).

- Torsten Rahn


On März 21, 2015, 9:52 nachm., Dávid Kolozsvári wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/123097/
> -----------------------------------------------------------
> 
> (Updated März 21, 2015, 9:52 nachm.)
> 
> 
> Review request for Marble.
> 
> 
> Repository: marble
> 
> 
> Description
> -------
> 
> The GroundOverlay was only resizable from its corners, this patch adds horizontal and vertical resize capability to it.
> Besides that, the corners are shown as nodes now, to let the user know that he can grab it.
> 
> 
> Diffs
> -----
> 
>   src/plugins/render/annotate/AnnotatePlugin.cpp cd5a334 
>   src/plugins/render/annotate/GroundOverlayFrame.h d8eee53 
>   src/plugins/render/annotate/GroundOverlayFrame.cpp d96bf4a 
>   src/plugins/render/annotate/SceneGraphicsItem.h cac5e4b 
> 
> Diff: https://git.reviewboard.kde.org/r/123097/diff/
> 
> 
> Testing
> -------
> 
> It was tested by me and works fine.
> 
> 
> Thanks,
> 
> Dávid Kolozsvári
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/marble-devel/attachments/20150322/5c20f3f5/attachment.html>


More information about the Marble-devel mailing list