ngraham accepted this revision. ngraham added a reviewer: broulik. ngraham added a comment.
The logic looks sane to me and the UI is good, but the time-to-empty/time-to-full values still don't always update correctly for me after a state change. I suspect this is something that @broulik will need to advise on. Maybe it's just not auto-updating itself because it was never needed to before this patch? REPOSITORY R120 Plasma Workspace REVISION DETAIL https://phabricator.kde.org/D24070 To: mthw, ngraham, #vdg, #plasma, broulik Cc: ndavis, broulik, plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, GB_2, ragreen, ZrenBot, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart