Review Request 125006: Add individual monitor for each cpu option for system load viewer

[Kåre Särs]

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

Review request for Plasma, David Edmundson and Martin Yrjölä.


Repository: kdeplasma-addons


Description
-------

When compact-bar monitos are selected the "show: CPUs separately" is enabled. When activated every cpu gets it's own monitor. The individual monitors try to fit in the space of two bars but when the number of cpus are too high the widht of the cpu monitors grow proportionally.


Diffs
-----

  applets/systemloadviewer/package/contents/config/main.xml 0edfd89 
  applets/systemloadviewer/package/contents/ui/GeneralSettings.qml a577b1a 
  applets/systemloadviewer/package/contents/ui/SystemLoadViewer.qml cb63fed 

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


Testing
-------

Running it on my own computer ;)

Issues:
1) When changing the CPUs separately setting the cpu column(s) size(s) are set to minimal and a restart of the applet is needed to get good loking layouts.
2) The tooltip does not fit all the cpu rows if the number of cpus are over 4 (6?).


Thanks,

Kåre Särs

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