Review Request: Patch to add free space indicator into device notifier plasmoid

martin.klapetek at gmail.com martin.klapetek at gmail.com
Sat May 9 02:29:04 CEST 2009


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

(Updated 2009-05-08 17:29:03.446499)


Review request for Plasma.


Changes
-------

Almost final version, the last thing to do is to make sizeHint in DeviceSpaceInfoDelegate count with font size and thus making the rect bigger/smaller. Also thanks to jackrabbit for his modifications.


Summary
-------

This is more like a proof of concept, for me to see if I'm on the right way with this one


Diffs (updated)
-----

  trunk/KDE/kdebase/workspace/plasma/applets/devicenotifier/CMakeLists.txt 965209 
  trunk/KDE/kdebase/workspace/plasma/applets/devicenotifier/devicenotifier.cpp 965209 
  trunk/KDE/kdebase/workspace/plasma/applets/devicenotifier/notifierdialog.h 965209 
  trunk/KDE/kdebase/workspace/plasma/applets/devicenotifier/notifierdialog.cpp 965209 
  trunk/KDE/kdebase/workspace/plasma/applets/devicenotifier/notifierview.cpp 965209 

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


Testing
-------


Thanks,

mck182



More information about the Plasma-devel mailing list