Review Request: Folderview : label when empty folder
Iamluc
luc at vieillescazes.net
Thu Jul 8 21:22:01 CEST 2010
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.kde.org/r/4519/
-----------------------------------------------------------
(Updated 2010-07-08 19:22:01.397461)
Review request for Plasma.
Changes
-------
New version using IconView::paintMessage() (no more signal/slot)
Fredrik, thanks for your help.
Simon, I'm sorry, I tried to reduce the size of the popup, but it didn't work.
In fact, the popup is visible before we know if there are items or not. And resizing it after it appears is really ugly !
Summary
-------
Hello,
This patch add a Plasma::Label with text "This folder is empty." in Folderview's Popup when the folder is empty.
The related bug number in kde bugzilla is 201542.
Luc
This addresses bug 201542.
https://bugs.kde.org/show_bug.cgi?id=201542
Diffs (updated)
-----
/trunk/KDE/kdebase/apps/plasma/applets/folderview/iconview.h 1145373
/trunk/KDE/kdebase/apps/plasma/applets/folderview/iconview.cpp 1145373
Diff: http://reviewboard.kde.org/r/4519/diff
Testing
-------
Delete/add files several times. The label disappears/appears correctly.
Screenshots
-----------
folderview empty folder
http://reviewboard.kde.org/r/4519/s/444/
Thanks,
Iamluc
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/plasma-devel/attachments/20100708/4b70cd22/attachment.htm
More information about the Plasma-devel
mailing list