Skip to content

applets/batterymonitor: Improve text and icon if neither batteries nor brightness controls are present

Fabian Vogt requested to merge work/fvogt/batteryicon3 into master

f9e39815 ("applets/batterymonitor: Improve for desktop systems") changed the plasmoid to use a plain icon instead of the CompactRepresentation if there are no batteries. While 471cc374 ("applets/batterymonitor: fix CompactRepresentation") brought the use of the CompactRepresentation back, the plain icon is still used.

For the case that there are no batteries and brightness controls available, it fell back to "battery", which has no symbolic variant. Change that to use the symbolic "video-display-brightness" as well.

BUG: 448520

CC @apol @ngraham

Maybe it should just say "Power Management" in this case?

Plasma < 5.24.90:

Screenshot_20220118_230521

Plasma 5.24.90:

Screenshot_20220118_230616

This MR:

Screenshot_20220118_231605

Edited by Fabian Vogt

Merge request reports