Review Request: Extend FolderView::configChanged() to reload configuration values
Rohan Garg
rohangarg at ubuntu.com
Sun Aug 22 19:07:53 CEST 2010
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/5049/
-----------------------------------------------------------
(Updated 2010-08-22 17:07:53.057046)
Review request for Plasma and Fredrik Höglund.
Changes
-------
Final patch to fix all the stuff i could find.Seems to be working after testing on my machine
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 1165986
Diff: http://reviewboard.kde.org/r/5049/diff
Testing (updated)
-------
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/20100822/17f5a67e/attachment.htm
More information about the Plasma-devel
mailing list