screensaver_pause plugin patch to support KDE, et al.

Bug #1071551 reported by Boscorama
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Pithos
New
Undecided
Unassigned

Bug Description

I know this app (thanks, btw) is targeted at GNOME but with the simple patch that is attached the screensaver_pause plugin will also work on KDE and any other windowing system that uses the freedesktop.org messages.

Revision history for this message
Boscorama (ppa-boscorama) wrote :
Revision history for this message
Chris Jowett (cryptk) wrote :

I was about to submit this exact same bug... it always irritated me that it didn't pause when my KDE screensaver activated, and I came up with the same patch (Great minds think alike) with the exception that I removed the line for org.gnome.ScreenSaver. I was worried that it would end up triggering the play/pause twice if they were both there (which would defeat the purpose).

That said, I don't have a gnome system to test that on, so it may or may not be an issue.

Revision history for this message
Chris Jowett (cryptk) wrote :

Nevermind on my previous worries... looking at the code more closely shows that the screensaver_pause plugin does indeed check the state returned by dbus, so it should not double-fire the playPause (well... technically it will, but it can handle that properly).

To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.