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

2010-09-15 Thread Rohan Garg
--- This is an automatically generated e-mail. To reply, visit: http://svn.reviewboard.kde.org/r/5049/ --- (Updated 2010-09-15 17:37:23.970844) Review request for Plasma and Fredrik H

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

2010-09-15 Thread Rohan Garg
--- 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

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

2010-09-14 Thread Rohan Garg
--- This is an automatically generated e-mail. To reply, visit: http://svn.reviewboard.kde.org/r/5049/ --- (Updated 2010-09-14 17:24:49.849536) Review request for Plasma and Fredrik H

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

2010-09-14 Thread Rohan Garg
--- This is an automatically generated e-mail. To reply, visit: http://svn.reviewboard.kde.org/r/5049/ --- (Updated 2010-09-14 16:39:31.059811) Review request for Plasma and Fredrik H

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

2010-08-23 Thread Sebastian Kügler
On Sunday 22 August 2010 19:44:36 Harald Sitter wrote: > /trunk/KDE/kdebase/apps/plasma/applets/folderview/folderview.cpp (Diff > revision 7) void FolderView::init() > 444 > const QList iconSizes = QList() << 16 << 22 << 32 << 48 << > 64 << 128; This is duplicated with configAccepted! > >

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

2010-08-22 Thread Harald Sitter
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/5049/#review7153 --- Some lines are a mystery to me. /trunk/KDE/kdebase/apps/plasma/apple

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

2010-08-22 Thread Rohan Garg
--- 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öglu

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

2010-08-22 Thread Rohan Garg
--- 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öglu

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

2010-08-17 Thread Rohan Garg
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/5049/ --- (Updated 2010-08-17 12:12:32.350700) Review request for Plasma and Fredrik Höglu

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

2010-08-17 Thread Rohan Garg
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/5049/ --- (Updated 2010-08-17 12:09:14.676873) Review request for Plasma and Fredrik Höglu

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

2010-08-17 Thread Rohan Garg
--- 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öglu

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

2010-08-17 Thread Rohan Garg
--- 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 FolderVie

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

2010-08-16 Thread Marco Martin
--- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/5049/#review7089 --- Ship it! this patch was really needed to make folderview correctly co