mthw added inline comments.

INLINE COMMENTS

> broulik wrote in batterymonitor.qml:56
> You still haven't addressed this. Don't just `return` when 
> `powermanagementDisabled`, I would still want to see my battery state when 
> this is the case?

OK, would you like to see:
MainText: Power management disabled
SubText: Battery at X%, HH:MM left
Or:
MainText: Battery at X% 
SubText:  HH:MM left
Or something else?

REPOSITORY
  R120 Plasma Workspace

REVISION DETAIL
  https://phabricator.kde.org/D24070

To: mthw, ngraham, #vdg, #plasma, broulik, ndavis
Cc: ndavis, broulik, plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, 
fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, ahiemstra, mart

Reply via email to