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

Kåre Särs kare.sars at iki.fi
Mon Nov 2 11:49:31 UTC 2015


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

(Updated Nov. 2, 2015, 11:49 a.m.)


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


Changes
-------

Add Layout.minimumWidth to prevent overlapping, remove a hardcoded pixelWidth, rebase on master and minor code cleanup


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 (updated)
-----

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

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


Testing
-------

Running it on my own computer ;)


Thanks,

Kåre Särs

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


More information about the Plasma-devel mailing list