Improve the notification when the ac adapter is plugged/unplugged.
Displaying direcly in them the time remaining to empty or fill the battery is quite more informative.
Details
Details
Diff Detail
Diff Detail
- Repository
- R122 Powerdevil
- Branch
- arcpatch-D20492
- Lint
No Linters Available - Unit
No Unit Test Coverage - Build Status
Buildable 10795 Build 10813: arc lint + arc unit
Comment Actions
That's my intent, we can yet talk about wording.
I can now move forward after recompiling everything...
Comment Actions
When AC is plugged:
When AC is unplugged:
I have noticed that sometimes the remaining time is not available.
In which case the percent is used, but I haven't tested this case properly.
Also I hesitate to always include the percent in those notifications.
Comment Actions
Also I'd like to display the duration naturally like "1 hour 30 minutes" instead of "1:30" as it does currently.
Could someone point me to the right Kcore/KLocale feature for this ?
I have done a little research but haven't found any, perhaps we are lacking the feature.
Comment Actions
I believe so, and that the plugged out event is notified by a sound by default. But I am not sure at the moment.
But with this feature they become quite more useful, especially the plugged out notification. It could be worse it to make it visible by default.
I personally dislike sound notifications and I suspect I am not the only one.
The feature is a piggyback feature from macOS.
Comment Actions
I think remaining time is way too unreliable for this. We first need a way to calculate a moving average before showing it i this context imho
Comment Actions
How come this be be good enough for the battery applet ? Isn't it more exposed in this context ?
The missing moving average seems to me like a second underlying issue.
The current remainTime is instantaneous true, but informative nonetheless albeit less accurate.
At least it would be coherent everywhere (applet and notification).
Still we don't need to rush this.
Creating a powerdevil bug/task about moving average battery level could be the outcome here.