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

Rohan Garg rohangarg at ubuntu.com
Sun Aug 22 19:05:40 CEST 2010


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

(Updated 2010-08-22 17:05:40.407310)


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 1165986 

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/20100822/0db3dd0e/attachment.htm 


More information about the Plasma-devel mailing list