[plasmashell] [Bug 449635] Battery applet doesn't list apps inhibiting sleep.

2022-02-08 Thread Lemuel Simon
https://bugs.kde.org/show_bug.cgi?id=449635

Lemuel Simon  changed:

   What|Removed |Added

 Resolution|--- |FIXED
 Status|REPORTED|RESOLVED
   Version Fixed In||5.24.0

--- Comment #16 from Lemuel Simon  ---
(In reply to Lemuel Simon from comment #15)
> Created attachment 146447 [details]
> Battery and Brightness applet show sleep inhibiting apps again.
> 
> I just updated to Plasma 5.24 today. Sleep inhibiting apps are now listed in
> the 'Battery and Brightness' applet as it did before. It works as expected,
> so I'll close the bug. @Ratijas, thanks for looking into the matter. You
> responses were quick and thorough, highly appreciated.

Uhh...I'm second-guessing here. Applets work but Plasma Engine Explorer still
shows nothing if the Inhibitions data source is selected under Power
Management. Separate issues, yeah?

-- 
You are receiving this mail because:
You are watching all bug changes.

[plasmashell] [Bug 449635] Battery applet doesn't list apps inhibiting sleep.

2022-02-08 Thread Lemuel Simon
https://bugs.kde.org/show_bug.cgi?id=449635

--- Comment #15 from Lemuel Simon  ---
Created attachment 146447
  --> https://bugs.kde.org/attachment.cgi?id=146447&action=edit
Battery and Brightness applet show sleep inhibiting apps again.

I just updated to Plasma 5.24 today. Sleep inhibiting apps are now listed in
the 'Battery and Brightness' applet as it did before. It works as expected, so
I'll close the bug. @Ratijas, thanks for looking into the matter. You responses
were quick and thorough, highly appreciated.

-- 
You are receiving this mail because:
You are watching all bug changes.

[plasmashell] [Bug 449635] Battery applet doesn't list apps inhibiting sleep.

2022-02-06 Thread Nate Graham
https://bugs.kde.org/show_bug.cgi?id=449635

Nate Graham  changed:

   What|Removed |Added

  Component|Battery Monitor |DataEngines
 CC||[email protected]

--- Comment #14 from Nate Graham  ---
> No, I can assure you there were no plans on removing inhibitions from the 
> «Battery and Brightness»
> applet. This is most certainly a bug is our data sources on backend part.
What exactly is the bug?

-- 
You are receiving this mail because:
You are watching all bug changes.

[plasmashell] [Bug 449635] Battery applet doesn't list apps inhibiting sleep.

2022-02-06 Thread Lemuel Simon
https://bugs.kde.org/show_bug.cgi?id=449635

--- Comment #13 from Lemuel Simon  ---
Created attachment 146375
  --> https://bugs.kde.org/attachment.cgi?id=146375&action=edit
PowerManagementItem.qml for Plasma 5.23.90

OK, I've attached the requested file. Sorry for the late response.

-- 
You are receiving this mail because:
You are watching all bug changes.

[plasmashell] [Bug 449635] Battery applet doesn't list apps inhibiting sleep.

2022-02-06 Thread ratijas
https://bugs.kde.org/show_bug.cgi?id=449635

--- Comment #12 from ratijas  ---
Created attachment 146372
  --> https://bugs.kde.org/attachment.cgi?id=146372&action=edit
«Display Configuration» and «Battery and Brightness» applets are fine

I glanced at Display Configuration applet and its sources. Looks nice, uses the
the same "powermanagement" data engine indeed. The code is moderately horrible,
and should be fixed just like B&B applet was. Now I see where's that
`inhibitions[0]` thing comes from in B&B, huh…

Anyways, works for me (on git master) in both applets just fine, inhibitions
are all there. Try building plasma-workspace package from source and see if it
helps — are you able to do that without destroying your system?

Or, you know what, let's do this: find the PowerManagementItem.qml file in your
system — locate(1) utility is your friend, but usually it should be something
like
/usr/share/plasma/plasmoids/org.kde.plasma.battery/contents/ui/PowerManagementItem.qml
— and upload it here or as a snippet on GitLab, so I can compare which version
was that. Since inhibitions are present in your Display Configuration applet
(and thus in the data engine), I suspect this is the issue that was already
fixed in master.

-- 
You are receiving this mail because:
You are watching all bug changes.

[plasmashell] [Bug 449635] Battery applet doesn't list apps inhibiting sleep.

2022-02-06 Thread Lemuel Simon
https://bugs.kde.org/show_bug.cgi?id=449635

--- Comment #11 from Lemuel Simon  ---
(In reply to ratijas from comment #10)
> No, I can assure you there were no plans on removing inhibitions from the
> «Battery and Brightness» applet. This is most certainly a bug is our data
> sources on backend part.

Then 'Display Configuration' is reading another data source, as in, its not
using the backend?

-- 
You are receiving this mail because:
You are watching all bug changes.

[plasmashell] [Bug 449635] Battery applet doesn't list apps inhibiting sleep.

2022-02-06 Thread ratijas
https://bugs.kde.org/show_bug.cgi?id=449635

--- Comment #10 from ratijas  ---
> Display Configuration

I… didn't even knew such applet exists.

> This may not be a bug after all, but a 'questionable' redesign. I found this 
> out by mistake and Display Configuration applet is not visible by default. It 
> isn't as discoverable as when it was in the Battery Applet.

No, I can assure you there were no plans on removing inhibitions from the
«Battery and Brightness» applet. This is most certainly a bug is our data
sources on backend part.

-- 
You are receiving this mail because:
You are watching all bug changes.

[plasmashell] [Bug 449635] Battery applet doesn't list apps inhibiting sleep.

2022-02-06 Thread Lemuel Simon
https://bugs.kde.org/show_bug.cgi?id=449635

--- Comment #9 from Lemuel Simon  ---
Created attachment 146367
  --> https://bugs.kde.org/attachment.cgi?id=146367&action=edit
Vivaldi playing Youtube video, inhibiting sleep, shown in 'Display
Configuration' applet.

Apparently, this information is shown in the 'Display Configuration' applet
instead of the Battery Applet. The 'Wakelock' for Chromium shows up there while
playing a video (see screenshot). This may not be a bug after all, but a
'questionable' redesign. I found this out by mistake and Display Configuration
applet is not visible by default. It isn't as discoverable as when it was in
the Battery Applet.

~~Also, why is Presentation Mode in both the Battery Applet and the Display
Configuration applet, but the Wakelock only shown in the latter?~~

Should I close this bug?

-- 
You are receiving this mail because:
You are watching all bug changes.

[plasmashell] [Bug 449635] Battery applet doesn't list apps inhibiting sleep.

2022-02-05 Thread ratijas
https://bugs.kde.org/show_bug.cgi?id=449635

ratijas  changed:

   What|Removed |Added

 Status|NEEDSINFO   |REPORTED
 Resolution|WAITINGFORINFO  |---

--- Comment #8 from ratijas  ---
> That assumption is correct. I clicked on the entry and it didn't expand, so I 
> tried manually typing it in.

Yeah, that's pretty stupid on engine explorer side.  It should provide some UI
means to understand that data source is not connected, and show a  "Connect"
button instead of requiring manual typing or context menu.

// TODO for myself: overhaul the Plasma Engine Explorer UI/UX

-- 
You are receiving this mail because:
You are watching all bug changes.

[plasmashell] [Bug 449635] Battery applet doesn't list apps inhibiting sleep.

2022-02-05 Thread Lemuel Simon
https://bugs.kde.org/show_bug.cgi?id=449635

--- Comment #7 from Lemuel Simon  ---
(In reply to ratijas from comment #6)
> > I've just realized that this was to be done in Plasma Engine Explorer than 
> > Qt5 Dbus Viewer. My apologies.
> 
> I was asking for both, so you did everything right. Thank you!
> 
> > Plasma Engine Explorer, Inhibitions data source doesn't expand.
> 
> Just to clarify: I see you typed in "Inhibitions" in the source name, so I
> assume you explicitly "Request"ed it, and it still does not expand, correct?
> If so, then it rules out the possibility of a bug in applet frontend, and we
> gotta investigate data providers.

That assumption is correct. I clicked on the entry and it didn't expand, so I
tried manually typing it in.

-- 
You are receiving this mail because:
You are watching all bug changes.

[plasmashell] [Bug 449635] Battery applet doesn't list apps inhibiting sleep.

2022-02-05 Thread ratijas
https://bugs.kde.org/show_bug.cgi?id=449635

--- Comment #6 from ratijas  ---
> I've just realized that this was to be done in Plasma Engine Explorer than 
> Qt5 Dbus Viewer. My apologies.

I was asking for both, so you did everything right. Thank you!

> Plasma Engine Explorer, Inhibitions data source doesn't expand.

Just to clarify: I see you typed in "Inhibitions" in the source name, so I
assume you explicitly "Request"ed it, and it still does not expand, correct? If
so, then it rules out the possibility of a bug in applet frontend, and we gotta
investigate data providers.

-- 
You are receiving this mail because:
You are watching all bug changes.

[plasmashell] [Bug 449635] Battery applet doesn't list apps inhibiting sleep.

2022-02-05 Thread Lemuel Simon
https://bugs.kde.org/show_bug.cgi?id=449635

--- Comment #5 from Lemuel Simon  ---
Created attachment 146309
  --> https://bugs.kde.org/attachment.cgi?id=146309&action=edit
Plasma Engine Explorer, Inhibitions data source doesn't expand.

I've just realized that this was to be done in Plasma Engine Explorer than Qt5
Dbus Viewer. My apologies.

-- 
You are receiving this mail because:
You are watching all bug changes.

[plasmashell] [Bug 449635] Battery applet doesn't list apps inhibiting sleep.

2022-02-04 Thread Lemuel Simon
https://bugs.kde.org/show_bug.cgi?id=449635

--- Comment #4 from Lemuel Simon  ---
Created attachment 146294
  --> https://bugs.kde.org/attachment.cgi?id=146294&action=edit
ListInhibitions dbus method (fake inhibitor 'foo' 'bar')

The fake inhibitor shows up in Dbus Viewer as well. Still isn't reflected in
the battery applet.

-- 
You are receiving this mail because:
You are watching all bug changes.

[plasmashell] [Bug 449635] Battery applet doesn't list apps inhibiting sleep.

2022-02-04 Thread Lemuel Simon
https://bugs.kde.org/show_bug.cgi?id=449635

--- Comment #3 from Lemuel Simon  ---
Created attachment 146293
  --> https://bugs.kde.org/attachment.cgi?id=146293&action=edit
ListInhibitions dbus method

ListInhibitions method successfully shows Vivaldi (web browser) inhibit sleep
while playing a video.

-- 
You are receiving this mail because:
You are watching all bug changes.

[plasmashell] [Bug 449635] Battery applet doesn't list apps inhibiting sleep.

2022-02-04 Thread ratijas
https://bugs.kde.org/show_bug.cgi?id=449635

ratijas  changed:

   What|Removed |Added

 Resolution|--- |WAITINGFORINFO
 Status|REPORTED|NEEDSINFO

--- Comment #2 from ratijas  ---
I suppose this was fixed in
https://invent.kde.org/plasma/plasma-workspace/-/merge_requests/1377

Apart from that, it also happens to my laptop that after waking up from sleep
Plasma's data engine doesn't get any inhibitions to list. The problem might be
in the integration glue or in the underlying D-Bus services.

When you think inhibitions do exist but are not listed, please post screenshots
of the following:
- open of Plasma Engine Explorer with "powermanagement" data engine selected,
request/update "Inhibitions" data source and expand it in tree view;
- open Qt D-Bus Viewer, then:
  * switch to Session Bus tab;
  * search for org.freedesktop.PowerManagement.Inhibit service;
  * in the right pane navigate to object
org/kde/Solid/PowerManagement/PolicyAgent
  * expand the org.kde.Solid.PowerManagement.PolicyAgent interface (the one
with italic font)
  * click on ListInhibitions method.
  * for testing purposes, click on AddInhibition method, enter any non-empty
string in second and third arguments (app id and reason), and click OK. Few
seconds later verify that inhibition this 'fake' inhibition should appear in
the ListInhibitions's output.

-- 
You are receiving this mail because:
You are watching all bug changes.

[plasmashell] [Bug 449635] Battery applet doesn't list apps inhibiting sleep.

2022-02-04 Thread Lemuel Simon
https://bugs.kde.org/show_bug.cgi?id=449635

Lemuel Simon  changed:

   What|Removed |Added

 CC||[email protected]
   ||m

-- 
You are receiving this mail because:
You are watching all bug changes.

[plasmashell] [Bug 449635] Battery applet doesn't list apps inhibiting sleep.

2022-02-04 Thread Lemuel Simon
https://bugs.kde.org/show_bug.cgi?id=449635

--- Comment #1 from Lemuel Simon  ---
Created attachment 146290
  --> https://bugs.kde.org/attachment.cgi?id=146290&action=edit
Vivaldi playing music, inhibiting sleep, doesn't appear in applet.

-- 
You are receiving this mail because:
You are watching all bug changes.