Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Presentation mode in the battery monitor doesn't prevent the screensaver #4275

Closed
gasull opened this issue Sep 5, 2018 · 14 comments · Fixed by QubesOS/qubes-xscreensaver#4
Closed
Labels
C: desktop-linux-xfce4 Support for XFCE4 diagnosed Technical diagnosis has been performed (see issue comments). P: major Priority: major. Between "default" and "critical" in severity. r4.0-dom0-stable

Comments

@gasull
Copy link

gasull commented Sep 5, 2018

Qubes OS version:

R4.0

Affected component(s):


Steps to reproduce the behavior:

Expected behavior:

The battery monitor panel item has the option "Presentation mode". Clicking on the checkbox should prevent the screensaver from starting.

Actual behavior:

Checking the box for "Presentation mode" doesn't have any effect on the screensaver. It's activated after the usual interval.

General notes:


Related issues:

@andrewdavidwong
Copy link
Member

Which DE are you using?

@gasull
Copy link
Author

gasull commented Sep 6, 2018

Which DE are you using?

The default one: Xfce.

I think the only things I changed are:

  1. Applications > System Tools > Screensaver > Display Modes > Blank After: 9 minutes; ☑️ Lock Screen After: 1 minutes
  2. Applications > System Tools > Appearance > Style > Industrial

@esote
Copy link

esote commented Sep 8, 2018

I checked and I also have this issue with XFCE.

@tasket
Copy link

tasket commented Sep 20, 2018

FWIW, this feature (among many others not working in Xfce) does work in KDE5. I should start making a list...

@imme-emosol
Copy link

What the "presentation mode" could do, probably through dom0;
enable presentation mode: xscreensaver-command -exit
disable presentation mode: xscreensaver && disown

@liketechnik
Copy link

Any update on this? Or a hint how to apply above "fix" on my local system?

@andrewdavidwong andrewdavidwong added the P: default Priority: default. Default priority for new issues, to be replaced given sufficient information. label Sep 29, 2019
@tabbyrobin
Copy link

@liketechnik I'm not sure i know how to get the xfce feature to work, but you might be able to get desired behavior (ability to watch movies without interruption...etc) by settings the xscreensaver settings directly. See here: https://www.jwz.org/xscreensaver/faq.html#dvd

When you want to watch a movie, fire up xscreensaver-demo and select Mode: Disable Screen Saver from the option menu, which means not to blank the screen at all. When you're done watching the movie, re-select your previous mode.
[...] If you are trying to use XFCE's "Presentation Mode"... don't. It breaks everything. Use XScreenSaver's "Disable Screen Saver" mode instead.

I haven't done this myself so can't guarantee how well it works. They also suggest two player-specific options (mplayer and vlc), which i assume WILL NOT work in qubes because they can't, and shouldn't, access dom0.

For some context/background, there seem to be a lot of problems caused by using xfce4-power-manager and xscreensaver (especially together). They compete/conflict.

For anyone coming across this thread and for future documentation, here's some links providing context:
https://forum.xfce.org/viewtopic.php?id=8154
https://bugzilla.xfce.org/show_bug.cgi?id=10205
https://bugs.launchpad.net/ubuntu/+source/xscreensaver/+bug/924278
https://www.jwz.org/xscreensaver/faq.html#dvd
https://www.jwz.org/xscreensaver/faq.html#no-blank
https://docs.xfce.org/xfce/xfce4-power-manager/preferences

And a (non-exhaustive) list of related Qubes issues...

#3590
#3674
#3727
#3898

@tlaurion
Copy link
Contributor

tlaurion commented Oct 17, 2020

@fepitre @marmarta @marmarek : this is sadly a missing feature and is totally giving a bad image to QubesOS since expected to work, the option being proposed but broken...

Mitigation is to turn off the screensaver. But if you do presentations... This is exactly the kind of stuff you forget to do prior of a presentation when focused on the task. You remember it doesn't work, but forget to apply the mitigation prior of starting to speak. Please fix this...

@andrewdavidwong andrewdavidwong added needs diagnosis Requires technical diagnosis from developer. Replace with "diagnosed" or remove if otherwise closed. P: major Priority: major. Between "default" and "critical" in severity. and removed P: default Priority: default. Default priority for new issues, to be replaced given sufficient information. labels Oct 18, 2020
@andrewdavidwong
Copy link
Member

We have a similar thread about this on the forum. There, I wrote:

Lately, I’ve found that if I just want to watch a movie or something without the screen locking or turning off, it works to make these two changes:

  1. XScreenSaver Settings -> Display Modes -> Mode -> Disable Screen Saver
  2. XScreenSaver Settings -> Advanced -> (uncheck) Power Management Enabled

I usually leave the XScreenSaver Settings window open in the background so that I don’t forget to change the settings back afterward. 🙂

It's not exactly the same as being able to toggle presentation mode, but some people might find it helpful as a workaround in many situations.

@burningrooftop
Copy link

This is a solution which works for me (ie. the Presentation Mode check box in power manager stops xscreensaver from activating):

Use System Tools > Settings Editor to add a new setting:

Property: /xfce4-power-manager/heartbeat-command
Type: String
Value: xscreensaver-command -disable

I assume you could also use xfconf-query to add the setting, but I haven't tried that myself.

See this bug report for details. The heartbeat-command setting doesn't seem to be well documented.

@rustybird
Copy link

rustybird commented Dec 17, 2020

Property: /xfce4-power-manager/heartbeat-command
Type: String
Value: xscreensaver-command -disable

Neat! But the xscreensaver-command argument should be -deactivate (there is no -disable).

I assume you could also use xfconf-query to add the setting, but I haven't tried that myself.

This works:

$ xfconf-query -c xfce4-power-manager -p /xfce4-power-manager/heartbeat-command -n -t string -s 'xscreensaver-command -deactivate'

For added safety, I also automatically turn Presentation Mode off again whenever the screen is locked (by hotkey or anything else that invokes the xflock4 script):

$ xfconf-query -c xfce4-session -p /general/LockCommand -n -t string -s 'eval xfconf-query -c xfce4-power-manager -p /xfce4-power-manager/presentation-mode -s false & xscreensaver-command -lock'

@burningrooftop
Copy link

You are correct of course - it is -deactivate. I like the idea of disabling presentation mode when the screen is locked too - I'll try that. Thanks for the feedback.

@qubesos-bot
Copy link

Automated announcement from builder-github

The package xscreensaver-5.45-4.fc25 has been pushed to the r4.0 testing repository for dom0.
To test this update, please install it with the following command:

sudo qubes-dom0-update --enablerepo=qubes-dom0-current-testing

Changes included in this update

@andrewdavidwong andrewdavidwong removed the needs diagnosis Requires technical diagnosis from developer. Replace with "diagnosed" or remove if otherwise closed. label Feb 11, 2021
@andrewdavidwong andrewdavidwong added the diagnosed Technical diagnosis has been performed (see issue comments). label Feb 11, 2021
@qubesos-bot
Copy link

Automated announcement from builder-github

The package xscreensaver-5.45-4.fc25 has been pushed to the r4.0 stable repository for dom0.
To install this update, please use the standard update command:

sudo qubes-dom0-update

Or update dom0 via Qubes Manager.

Changes included in this update

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C: desktop-linux-xfce4 Support for XFCE4 diagnosed Technical diagnosis has been performed (see issue comments). P: major Priority: major. Between "default" and "critical" in severity. r4.0-dom0-stable
Projects
None yet
Development

Successfully merging a pull request may close this issue.