| Summary: | Widget always claims everything is blocking both sleep and screen locking, which can be inaccurate | ||
|---|---|---|---|
| Product: | [Plasma] plasmashell | Reporter: | Juan Simón <decedion> |
| Component: | Power and Battery widget | Assignee: | Kai Uwe Broulik <kde> |
| Status: | RESOLVED FIXED | ||
| Severity: | normal | CC: | bugseforuns, jpetso, kde, kdebugs.81do7, mjrixnertx+kde, nate, plasma-bugs-null, robby.engelmann, tneo, trmdi, vikts |
| Priority: | NOR | Keywords: | usability |
| Version First Reported In: | 5.21.4 | ||
| Target Milestone: | 1.0 | ||
| Platform: | Arch Linux | ||
| OS: | Linux | ||
| Latest Commit: | https://invent.kde.org/plasma/powerdevil/-/commit/83f685b79cc31fb5aae03c16285da3f05907a47d | Version Fixed/Implemented In: | 6.6.0 |
| Sentry Crash Report: | |||
|
Description
Juan Simón
2020-03-03 14:33:15 UTC
Yeah, it's possible to inhibit system sleep but not display sleep/dimming. Elisa should do that. *** Bug 421522 has been marked as a duplicate of this bug. *** In my opinion at the very least, an audio player should not be inhibiting screen locking. > When Elisa plays music doesn't allow the monitor to be turned off.
Hmm... I can't reproduce it. My laptop screen is still turned off while the music is playing, sleep is inhibited.
I am using the latest git version. Am I missing anything here?
Are you sure, or did you just read the label in battery monitor? It will always say "screen locking" because it doesn't know what inhibition an app exactly requested. I also can't reproduce on neon unstable. The screen of my laptop is turned off during playback. Yes, it was based on the description from the battery monitor applet. Sorry. Ah, we should make the text show the specific inhibitions rather than a generic message. *** Bug 487118 has been marked as a duplicate of this bug. *** *** Bug 500725 has been marked as a duplicate of this bug. *** *** Bug 509400 has been marked as a duplicate of this bug. *** A relevant merge request has been created at https://invent.kde.org/plasma/powerdevil/-/merge_requests/598 (and is about to be merged) Git commit 83f685b79cc31fb5aae03c16285da3f05907a47d by Jakob Petsovits. Committed on 09/01/2026 at 17:00. Pushed by jpetso into branch 'master'. applets/batterymonitor: Redesign inhibition list again Highlights at a glance: * Phrasings distinguish between sleep and screen locking * Remove some labels, so that the user focuses on what matters Following commit f9c81566, the PowerDevil service provides enough information to show whether an inhibition is blocking just sleep or just screen locking, or both. Introducing this distinction complicated label phrasings though, so we also had to tweak the UI to remain straightforward enough. As per Nate's suggested design, the indicator text at the top is removed for simplicity. The "Manually Block" is once again a toggle, now at the top of the inhibition section. We also get rid of the heading "N applications are currently blocking sleep [and/or screen locking]", which used to appear when multiple inhibitions were requested. In order to tell whether sleep and/or screen locking are currently inhibited, the user therefore has to look at all the remaining individual list items. However, this is generally acceptable as more than a small handful of inhibition list items are exceedingly rare. (An accessible status description continues to summarize the section.) In return, the section as a whole is tidier, generally with less text to read, which makes it more likely that the user will actually try to parse the remaining items. M +104 -167 applets/batterymonitor/InhibitionItem.qml M +1 -1 applets/batterymonitor/PopupDialog.qml M +1 -1 applets/batterymonitor/plugin/inhibitioncontrol.cpp M +2 -2 applets/batterymonitor/plugin/inhibitioncontrol.h https://invent.kde.org/plasma/powerdevil/-/commit/83f685b79cc31fb5aae03c16285da3f05907a47d |