Add a show label option to the pie chart face
AbandonedPublic

Authored by davidedmundson on May 13 2020, 8:57 AM.

Details

Reviewers
mart
Group Reviewers
Plasma
Summary

To control the large "20%" text in the middle of a pie which can be
obnoxious in the panel

Diff Detail

Repository
R111 KSysguard Library
Branch
showLabelOnPie
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 26819
Build 26837: arc lint + arc unit
davidedmundson created this revision.May 13 2020, 8:57 AM
Restricted Application added a project: Plasma. · View Herald TranscriptMay 13 2020, 8:57 AM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
davidedmundson requested review of this revision.May 13 2020, 8:57 AM
mart accepted this revision.May 14 2020, 10:53 AM
This revision is now accepted and ready to land.May 14 2020, 10:53 AM
davidedmundson abandoned this revision.May 14 2020, 10:56 AM

Arjen suggested in a PM that we already have the concept of "total sensor" being optional or not. No point hiding it at a UI level when we should be doing it at a data level.