D20492: [WIP] Display time remaining to fill/empty the battery in the notification when the ac adapter is plugged/unplugged.

[Nathaniel Graham]

ngraham added inline comments.

INLINE COMMENTS

> powerdevilcore.cpp:635
> +            if (m_batteryRemainingTime > 0) {
> +                detail = i18n("You need %s to fill the battery", 
> KFormat().formatDuration(m_batteryRemainingTime, KFormat::HideSeconds));
> +            } else {

`Remaining charging time: %s`

> powerdevilcore.cpp:638
> +                int currentPercent = currentChargePercent();
> +                detail = i18n("Your battery charge level is %s %", 
> currentPercent);
> +            }

`Charge level: %s%`

> powerdevilcore.cpp:646
> +        if (m_batteryRemainingTime > 0) {
> +            detail = i18n("You have %s autonomy left on battery", 
> KFormat().formatDuration(m_batteryRemainingTime, KFormat::HideSeconds));
> +        } else {

`Remaining battery life: %s`

> powerdevilcore.cpp:649
> +            int currentPercent = currentChargePercent();
> +            detail = i18n("Your battery charge level is %s %", 
> currentPercent);
> +        }

`Charge level: %s%`

REPOSITORY
  R122 Powerdevil

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

To: meven, #plasma, broulik
Cc: ngraham, plasma-devel, jraleigh, GB_2, ragreen, Pitel, ZrenBot, lesliezhai, 
ali-mohamed, jensreuterberg, abetts, sebas, apol, mart