From kde-panel-devel Sat Nov 28 09:55:39 2009 From: "Yuen Hoe Lim" Date: Sat, 28 Nov 2009 09:55:39 +0000 To: kde-panel-devel Subject: Re: Review Request: Folderview popups should inherit file preview Message-Id: <20091128095539.17613.94889 () localhost> X-MARC-Message: https://marc.info/?l=kde-panel-devel&m=125940216607840 ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://reviewboard.kde.org/r/2286/ ----------------------------------------------------------- (Updated 2009-11-28 09:55:39.107970) Review request for Plasma and Fredrik Höglund. Summary (updated) ------- 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 :) (I know it's feature/string freeze now, but this is rectification of faulty behaviour - ie bugfix, so it can be accepted right? This itch happens to bother me somewhat :) 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 _______________________________________________ Plasma-devel mailing list Plasma-devel@kde.org https://mail.kde.org/mailman/listinfo/plasma-devel