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

Rohan Garg rohangarg at ubuntu.com
Wed Sep 15 19:24:40 CEST 2010


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

(Updated 2010-09-15 17:24:39.880260)


Review request for Plasma and Fredrik Höglund.


Changes
-------

More fixes for configChanged as pointed out by notmart


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 1173894 

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


Testing
-------

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


More information about the Plasma-devel mailing list