Review Request: Don't use a custom scrollbar widget in the systemtray job viewer
Marco Martin
notmart at gmail.com
Tue Dec 23 19:00:02 CET 2008
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://reviewboard.vidsolbach.de/r/313/
-----------------------------------------------------------
Review request for Plasma.
Summary
-------
This makes the job viewer extenderitem in the systemtray to use just the standard scrollbar widget instead of a custom one and layouts and plasma labels to display the info.
I think it's a pretty important thing to not make custom svg widget proliferate, otherwise theming will become a real mess (a different scrollbar for each possible label disposition would be really bad).
and a thing like that should be done like now or never :)
Diffs
-----
/home/kde/trunk/KDE/kdebase/plasma/applets/systemtray/ui/jobwidget.h
/home/kde/trunk/KDE/kdebase/plasma/applets/systemtray/ui/jobwidget.cpp
Diff: http://reviewboard.vidsolbach.de/r/313/diff
Testing
-------
tried to start some jobs, it seems to both look and behave in the exact way as before
Thanks,
Marco
More information about the Plasma-devel
mailing list