Review Request 128136: alignement for systemmonitoring applets

Andreas Kainz kainz.a at gmail.com
Thu Jun 9 11:58:31 UTC 2016


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/128136/
-----------------------------------------------------------

Review request for Plasma.


Repository: plasma-workspace


Description
-------

1. change the minimum size and the min width in the panel
2. speedLabels are now right align to the label cause you show the number which is drown on the right part of the diagramm 
3. round to 14 % instead of 14,14 % for cpu load

still missing:
a. hide the heading if the desktop widget has to less space for the rows
b. better friendlyName for ram, network
c. remove the gray line from the chart (0-line) at least when the applet was shown on the panel
d. don't show 0 KB/s or 0 B, ...

shedule
thanks for your review
if you know how I can fix one of the missing things, let me know
tiny improvements when missing stuff is finished.


Diffs
-----

  applets/systemmonitor/common/contents/ui/Applet.qml fffd8e0 
  applets/systemmonitor/common/contents/ui/DoublePlotter.qml 738ffbc 
  applets/systemmonitor/common/contents/ui/SinglePlotter.qml 987fd2c 
  applets/systemmonitor/cpu/contents/ui/cpu.qml 4ebbf92 

Diff: https://git.reviewboard.kde.org/r/128136/diff/


Testing
-------

on the desktop and the panel


Thanks,

Andreas Kainz

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20160609/f2524e89/attachment.html>


More information about the Plasma-devel mailing list