[Marble-devel] Re: Review Request: Option to delete theme

[Daniel Marth]

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

(Updated 2010-12-26 22:12:38.961728)


Review request for marble.


Changes
-------

Patch now applies. Preview images weren't removed if they had an other type than PNG, this is fixed now too.


Summary
-------

Adds an option to delete a theme if it is located in the home directory of the user. Sub-directories and the theme directory are removed if there is no other data in them. The mapwizard-patch is required to apply this patch.

This patch was created for a GCI task:
http://www.google-melange.com/gci/task/show/google/gci2010/kde/t129258692177

Update: Private functions are now in the private class.


Diffs (updated)
-----

  /trunk/KDE/kdeedu/marble/src/lib/MarbleThemeSelectView.h 1209452 
  /trunk/KDE/kdeedu/marble/src/lib/MarbleThemeSelectView.cpp 1209452 

Diff: http://svn.reviewboard.kde.org/r/6180/diff


Testing
-------


Thanks,

Daniel

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