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

Rohan Garg rohangarg at ubuntu.com
Mon Aug 16 20:13:07 CEST 2010


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

(Updated 2010-08-16 18:13:06.548619)


Review request for Plasma and Fredrik Höglund.


Changes
-------

Define a few variables


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 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/b8bda2c8/attachment.htm 


More information about the Plasma-devel mailing list