D21690: Only show system total cpu usage by default on high core count machines

Robin Appelman noreply at phabricator.kde.org
Sun Jun 9 19:22:11 BST 2019


rappelman updated this revision to Diff 59460.
rappelman added a comment.


  The reason for `isSourceValid` was for to allow getting the number of sources/cores, I didn't want to require changing all the other plasmoids by fully replacing the `onSourceAdded`/`addSource` logic.
  
  I've updated it to always show the system total and removed the `isSourceValid` logic

REPOSITORY
  R120 Plasma Workspace

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D21690?vs=59445&id=59460

REVISION DETAIL
  https://phabricator.kde.org/D21690

AFFECTED FILES
  applets/systemmonitor/common/contents/ui/Applet.qml
  applets/systemmonitor/common/contents/ui/ConfigGeneral.qml
  applets/systemmonitor/cpu/contents/ui/cpu.qml
  applets/systemmonitor/cpu/contents/ui/cpuConfig.qml

To: rappelman, #plasma, #vdg, davidedmundson
Cc: davidedmundson, plasma-devel, LeGast00n, ericadams, jraleigh, GB_2, ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20190609/d1e59ed8/attachment-0001.html>


More information about the Plasma-devel mailing list