Review Request: bug fixes for the system-monitor applet

Michel Lafon-Puyo michel.lafonpuyo at gmail.com
Tue May 11 10:14:21 CEST 2010


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

Review request for Plasma.


Summary
-------

As my first work for KDE, I would like to add some power management monitoring features to the system-monitor applet. Monitoring the cpu clock frequency when using the on-demand or the conservative governor could be useful (at least it is for me :)).

Before going further in this development I tried to fix some little bugs with the applet. Here is the list of the changes this patch introduces:
- the size of the applets when used in the panel and when no item is monitored was (0,0) but an icon were displayed and it overlapped the other applets (SM::Applet::CheckGeometry())
- set the preferred height to MINIMUM when no item is monitored (SM::Applet::displayNoAvailableSources()). When all the meters of an applet were removed at once, the size were unchanged and a big icon could be displayed.
- clear the content of the tooltip when nothing has to be displayed (SM::Applet::toolTipAboutToShow())
- when one or many items were set "unmonitored", the corresponding widgets were not correctly deleted (SM::Applet::deleteMeters())
- the removal of the layout and the meters were done on SM::Applet::connectToEngine(), I moved that part in a new method SM::Applet::removeLayout() that can be called more easily. This method is called by the applets on configuration change to achieve a clean update. 
- to be consistent with the other applets, the HDD applet has been changed to *not* populate the configuration with the list of the mounted volumes when there is no more item to monitor. Nevertheless, on the first launch (no configuration is present), the behaviour has not changed and the configuration is still populated with the list of the mounted volumes.

Additionnally, the HDD applet didn't use the SM::Applet to manage its meters. So I replaced the list of SM::Plotter (m_plotters) by a list of QGraphicsWidget (m_meters) and modified HDD to take advantage of the meters management already implemented in the SM::Applet class (in particular the removal of the meters/widgets on configuration change as mentioned above). 

Note: I was unable to find any bug reports corresponding to the fixes above. Should I create them myself?


Diffs
-----

  /trunk/KDE/kdebase/workspace/plasma/generic/applets/system-monitor/applet.h 1125154 
  /trunk/KDE/kdebase/workspace/plasma/generic/applets/system-monitor/applet.cpp 1125154 
  /trunk/KDE/kdebase/workspace/plasma/generic/applets/system-monitor/cpu.h 1125154 
  /trunk/KDE/kdebase/workspace/plasma/generic/applets/system-monitor/cpu.cpp 1125154 
  /trunk/KDE/kdebase/workspace/plasma/generic/applets/system-monitor/hdd.h 1125154 
  /trunk/KDE/kdebase/workspace/plasma/generic/applets/system-monitor/hdd.cpp 1125154 
  /trunk/KDE/kdebase/workspace/plasma/generic/applets/system-monitor/hwinfo.h 1125154 
  /trunk/KDE/kdebase/workspace/plasma/generic/applets/system-monitor/hwinfo.cpp 1125154 
  /trunk/KDE/kdebase/workspace/plasma/generic/applets/system-monitor/net.cpp 1125154 
  /trunk/KDE/kdebase/workspace/plasma/generic/applets/system-monitor/ram.cpp 1125154 
  /trunk/KDE/kdebase/workspace/plasma/generic/applets/system-monitor/temperature.cpp 1125154 

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


Testing
-------

Basic testing


Thanks,

Michel



More information about the Plasma-devel mailing list