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

[Torsten Rahn]

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



/trunk/KDE/kdeedu/marble/src/plugins/render/positionmarker/PositionMarker.cpp
<http://svn.reviewboard.kde.org/r/6232/#comment10479>

    please write the setting for the m_useCustomCursor into a settings object.
    Same with the Url to the Pixmap.


- 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/e5a8d2a0/attachment.htm