[Battery Monitor] Call battery capacity "Capacity degradation" for better clarity
ClosedPublic

Authored by ngraham on Aug 19 2019, 7:12 PM.

Details

Summary

Companion patch to D23270

The same logic explained in D23270 applies here: using the text "Capacity degradation"
is much clearer than "capacity" when referring to how much of the battery's original
capacity is remaining due to degradation.

Test Plan

Diff Detail

Repository
R120 Plasma Workspace
Branch
capacity-to-degradation (branched from master)
Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 15349
Build 15367: arc lint + arc unit
ngraham created this revision.Aug 19 2019, 7:12 PM
Restricted Application added a project: Plasma. · View Herald TranscriptAug 19 2019, 7:12 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
ngraham requested review of this revision.Aug 19 2019, 7:12 PM
ngraham retitled this revision from [Battery Monitor] Call battery capacity "degredation" for better clarity to [Battery Monitor] Call battery capacity "degradation" for better clarity.
broulik accepted this revision.Aug 20 2019, 6:43 AM
broulik added inline comments.
applets/batterymonitor/package/contents/ui/logic.js
45–46

I think this needs some explanatory context

This revision is now accepted and ready to land.Aug 20 2019, 6:43 AM
filipf accepted this revision.Aug 20 2019, 8:23 AM
filipf added a subscriber: filipf.

+1, same reasoning as in D23270#514978.

ngraham updated this revision to Diff 64123.Aug 20 2019, 2:00 PM
ngraham marked an inline comment as done.

Add context

aspotashev added a subscriber: aspotashev.EditedAug 20 2019, 2:36 PM

I disagree with this change, see D23270#515102

ngraham updated this revision to Diff 64142.Aug 20 2019, 5:42 PM

Use the superior agreed-upon string "Capacity degradation"

ngraham edited the test plan for this revision. (Show Details)Aug 20 2019, 5:44 PM
ngraham retitled this revision from [Battery Monitor] Call battery capacity "degradation" for better clarity to [Battery Monitor] Call battery capacity "Capacity degradation" for better clarity.
ngraham edited the summary of this revision. (Show Details)
This revision was automatically updated to reflect the committed changes.