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