Review Request: Extend FolderView::configChanged() to reload configuration values
Rohan Garg
rohangarg at ubuntu.com
Tue Sep 14 18:39:31 CEST 2010
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://svn.reviewboard.kde.org/r/5049/
-----------------------------------------------------------
(Updated 2010-09-14 16:39:31.059811)
Review request for Plasma and Fredrik Höglund.
Changes
-------
After much poking around and thanks to notmart and aseigo, finally implemented it :D
Summary
-------
Extend FolderView::configChanged() such that all configuration options are read there on FolderView start. This will ensure that configuration changes made "behind its back" (e.g. by the Desktop Scripting) will take effect.
Diffs (updated)
-----
/trunk/KDE/kdebase/apps/plasma/applets/folderview/folderview.cpp 1173894
Diff: http://svn.reviewboard.kde.org/r/5049/diff
Testing
-------
Testing done, seems to be OK at my end
Thanks,
Rohan
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/plasma-devel/attachments/20100914/7dd2e448/attachment.htm
More information about the Plasma-devel
mailing list