When the battery in a non-power supply battery drops below 10% (configurable)
a notification is shown to the user so he or she is not surprised when the
device suddenly stops working.
BUG: 102097
FIXED-IN: 5.7.0
Details
Details
- Reviewers
mart - Group Reviewers
Plasma: Design Plasma - Commits
- R122:550bdc40d4e8: Merge branch 'broulik/notifyPeripherals'
So far I have only tested this with my mouse on a desktop computer,
starting kded5 brings the notification. Needs more testing with a laptop so it
didn't break the default battery handling
Diff Detail
Diff Detail
- Repository
- R122 Powerdevil
- Lint
Automatic diff as part of commit; lint not applicable. - Unit
Automatic diff as part of commit; unit tests not applicable.
| PowerDevilSettings.kcfg | ||
|---|---|---|
| 35 | Given the experience with my trackball: I think I used it for another half year once it dropped below 10 % | |
| daemon/powerdevilcore.cpp | ||
| 508 | In English that's fine, but in e.g. a German translation it would be horrible to suggest to charge a battery. (For non German speakers: a "Batterie" is the non-rechargeable one, an "Akku" is the rechargeable one). Given that it needs i18nc | |
| kcmodule/global/GeneralPage.cpp | ||
| 123 | new connect syntax? | |
| PowerDevilSettings.kcfg | ||
|---|---|---|
| 35 | My trackball has the peculiarity of dropping in 10% or 20% steps, so it might not even trigger then notification when it goes from 20% to 0%. Not sure how to proceed there. | |
| daemon/powerdevilcore.cpp | ||
| 508 | I never explained "Battery" anywhere and they always got it right. (I could show a different text depending on whether it reports the battery as "rechargeable", not sure how reliable that property is, though ;)) | |
| kcmodule/global/GeneralPage.cpp | ||
| 123 | I wanted to but the signature doesn't match so I would have had to cramp a lambda in there. Might change that at a later point (where I'll adjust the other connects too) | |
| daemon/powerdevilcore.cpp | ||
|---|---|---|
| 508 | Default german translation for battery is Akku, so obviously no context required | |
