Review Request: Folderview popups should inherit file preview settings

[Yuen Hoe Lim]

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

Review request for Plasma and Fredrik Höglund.


Summary
-------

Currently the folderview on-hover popups always previews (only) images no matter what the preview settings of the parent folderview applet is. I saw a 'TODO' comment in the code that says popups should inherit file preview settings from the parent, so I assumed this is the right / desired behavior and implemented it :)

I think my approach is sensible, and it would also accommodate allowing the folderview and the popup to have different file preview settings if we ever need that in future. Still, this is my first time ever staring at folderview code, so if I'm doing something unspeakably wrong, or if there's a better way to do this, please let me know :)


Diffs
-----

  /trunk/KDE/kdebase/apps/plasma/applets/folderview/folderview.cpp 1055216 
  /trunk/KDE/kdebase/apps/plasma/applets/folderview/iconview.h 1055216 
  /trunk/KDE/kdebase/apps/plasma/applets/folderview/iconview.cpp 1055216 
  /trunk/KDE/kdebase/apps/plasma/applets/folderview/popupview.h 1055216 
  /trunk/KDE/kdebase/apps/plasma/applets/folderview/popupview.cpp 1055216 

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


Testing
-------

Tested on trunk. Works AFAIK.


Thanks,

Yuen Hoe