Review Request: Extend FolderView::configChanged() to reload configuration values

Rohan Garg rohangarg at ubuntu.com
Mon Aug 16 16:31:06 CEST 2010


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

Review request for Plasma and Fredrik Höglund.


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
-----

  /trunk/KDE/kdebase/apps/plasma/applets/folderview/folderview.cpp 1164320 

Diff: http://reviewboard.kde.org/r/5049/diff


Testing
-------

Im looking for testers at the moment, but i know for a fact that after applying the patch that kdebase compiles.Will report back when i can get hold of testers and see if it actually works


Thanks,

Rohan

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/plasma-devel/attachments/20100816/ae3b9286/attachment-0001.htm 


More information about the Plasma-devel mailing list