ngraham added a comment.
In D23270#515330 <https://phabricator.kde.org/D23270#515330>, @aspotashev wrote: > In D23270#515293 <https://phabricator.kde.org/D23270#515293>, @ngraham wrote: > > > At least, that's how it was for me. I did not understand what "Capacity:" was referring to until I started browsing the code. "Capacity" is an ambiguous and non-descriptive term that does not really communicate the concept we're trying to get across. > > > How about this: "Capacity: 79% (degraded by 21%)"? > Or keep the label as is and add a link "What's this?" that would show a popup with an in-depth description of battery capacity, degradation, why it happens and what to do about it. Hmm, both suggestions seem to be designed to explain what "capacity" means rather than replacing it with something that doesn't need explanation at all. I don't really see the point of keeping the word "Capacity" visible when we can replace it with something that doesn't need additional explanation. Also, I don't think QML stuff do "What's This?" text, and even if it could I wouldn't want to use it because the "What's This?" feature is poorly-discoverable and not a substitute for clarity in the user-visible strings. REPOSITORY R102 KInfoCenter BRANCH capacity-to-degradation (branched from master) REVISION DETAIL https://phabricator.kde.org/D23270 To: ngraham, meven, broulik, #vdg, #plasma, filipf Cc: aspotashev, filipf, plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, GB_2, ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
