D23270: [Energy] Put battery capacity info in battery section and call it "degradation"
Nathaniel Graham
noreply at phabricator.kde.org
Tue Aug 20 17:48:00 BST 2019
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
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20190820/b90a8782/attachment-0001.html>
More information about the Plasma-devel
mailing list