[Marble-devel] Re: Review Request: Make the Position Marker plugin configurable by adding a settings page

Torsten Rahn rahn at kde.org
Thu Dec 30 01:32:00 CET 2010


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

Ship it!


I have submitted this patch already. Please mark this reviewrequest as submitted and please correct the readSettings/writeSettings issue in a separate patch.

- Torsten


On 2010-12-29 23:46:50, Daniel Marth wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://svn.reviewboard.kde.org/r/6232/
> -----------------------------------------------------------
> 
> (Updated 2010-12-29 23:46:50)
> 
> 
> Review request for marble.
> 
> 
> Summary
> -------
> 
> Adds a configuration dialog to the position marker plugin to load a custom image as marker.
> GCI task: http://www.google-melange.com/gci/task/show/google/gci2010/kde/t129347413457
> 
> 
> Diffs
> -----
> 
>   /trunk/KDE/kdeedu/marble/src/plugins/render/positionmarker/CMakeLists.txt 1210125 
>   /trunk/KDE/kdeedu/marble/src/plugins/render/positionmarker/PositionMarker.h 1210125 
>   /trunk/KDE/kdeedu/marble/src/plugins/render/positionmarker/PositionMarker.cpp 1210125 
>   /trunk/KDE/kdeedu/marble/src/plugins/render/positionmarker/PositionMarkerConfigWidget.ui PRE-CREATION 
> 
> Diff: http://svn.reviewboard.kde.org/r/6232/diff
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Daniel
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/marble-devel/attachments/20101230/96558bfe/attachment.htm 


More information about the Marble-devel mailing list