[Marble-devel] Review Request 114732: StarsPlugin user interface refactoring

Torsten Rahn tackat at kde.org
Sun Dec 29 21:37:20 UTC 2013


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

Ship it!


Ship It!

- Torsten Rahn


On Dec. 29, 2013, 9:26 p.m., Illya Kovalevskyy wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/114732/
> -----------------------------------------------------------
> 
> (Updated Dec. 29, 2013, 9:26 p.m.)
> 
> 
> Review request for Marble and Torsten Rahn.
> 
> 
> Repository: marble
> 
> 
> Description
> -------
> 
> According to Google Code-In 2013 task (https://google-melange.appspot.com/gci/task/view/google/gci2013/5811309861928960):
> 
> Refactoring of the StarsPlugin is accomplished. Memory leaks fixed, UI layout (menu entries) improved.
> 
> 
> Diffs
> -----
> 
>   src/plugins/render/stars/StarsPlugin.h 662ab1a 
>   src/plugins/render/stars/StarsPlugin.cpp 9a6fd4d 
> 
> Diff: https://git.reviewboard.kde.org/r/114732/diff/
> 
> 
> Testing
> -------
> 
> All possible behaviours tested, unit-tests.
> 
> 
> File Attachments
> ----------------
> 
> Context menu
>   https://git.reviewboard.kde.org/media/uploaded/files/2013/12/29/f4880fe8-21d2-4270-8593-bf40ad7b8b53__Screen_Shot_2013-12-29_at_11.22.43_PM.png
> 
> 
> Thanks,
> 
> Illya Kovalevskyy
> 
>

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


More information about the Marble-devel mailing list