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

Update screen blanking #3065

Merged
merged 2 commits into from
Aug 22, 2023
Merged

Update screen blanking #3065

merged 2 commits into from
Aug 22, 2023

Conversation

jehlers42
Copy link
Contributor

re: issue #2959

Removed instructions on adjusting the blank interval via the config file. Internal discussion led to encouraging changes solely through raspi-config or the config GUI.

@aallan
Copy link
Contributor

aallan commented Aug 22, 2023

Closes #2959

@aallan aallan linked an issue Aug 22, 2023 that may be closed by this pull request
@aallan aallan merged commit 3e2c1a6 into bookworm Aug 22, 2023
1 check passed
@aallan aallan deleted the iss-2959 branch August 22, 2023 14:29
@aallan aallan restored the iss-2959 branch August 22, 2023 14:29
@aallan aallan deleted the iss-2959 branch August 22, 2023 14:29
@aallan aallan restored the iss-2959 branch August 22, 2023 14:29
@aallan aallan deleted the iss-2959 branch August 22, 2023 14:29
@aallan
Copy link
Contributor

aallan commented Aug 22, 2023

It's not Monday, but it feels like it sometimes… 🙄

Screenshot 2023-08-22 at 15 30 18

@lurch
Copy link
Contributor

lurch commented Aug 22, 2023

I just noticed that the text just beyond the change in this PR also talks about "sudo apt install xscreensaver" - we probably don't want that, as Bookworm / Wayland comes with a screensaver built in by default? (It displays a rotating-cube of the desktop)
(and I'd guess that a program named xscreensaver might not work on Wayland?)

ping @spl237 for thoughts / comments

@aallan
Copy link
Contributor

aallan commented Aug 22, 2023

(and I'd guess that a program named xscreensaver might not work on Wayland?)

I'd guess it will not work on Wayland. Probably should pull the reference in any case, it's not going to be needed, Wayland comes with it built in. @jehlers42 can you PR something to pull that ref. Thanks.

@jehlers42
Copy link
Contributor Author

Since that section mentions finding the xscreensaver app in Preferences, will the new built-in screensaver have a similar location for options relating to it or should I just remove the entire screensaver bit.

@aallan
Copy link
Contributor

aallan commented Aug 23, 2023

I'd lean towards just removing it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Screen-blanking section needs to be checked / updated
3 participants