Review Request: Corona::exportLayout

Chani Armitage chanika at gmail.com
Sat Sep 25 18:51:30 CEST 2010


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

Review request for Plasma.


Summary
-------

this adds exportLayout to corona, which saves a group of containments to a config file and deletes them from the main config.

Activity::close() becomes a lot shorter by calling it, like this: m_corona->exportLayout(external, m_containments.values());

I feel a bit out of it today though, so tell me if I've missed anything obvious...


Diffs
-----

  trunk/KDE/kdelibs/plasma/corona.cpp 1177115 
  trunk/KDE/kdelibs/plasma/corona.h 1177115 

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


Testing
-------

closing an activity while locked is perfectly safe now :)


Thanks,

Chani

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/plasma-devel/attachments/20100925/413ea5f5/attachment.htm 


More information about the Plasma-devel mailing list