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

dpms off not working properly on lock screen #7689

Closed
dotaxis opened this issue Sep 7, 2024 · 3 comments
Closed

dpms off not working properly on lock screen #7689

dotaxis opened this issue Sep 7, 2024 · 3 comments
Labels
bug Something isn't working

Comments

@dotaxis
Copy link

dotaxis commented Sep 7, 2024

Regression?

Yes

System Info and Version

027140b7315efe3cd2e7b78fa608bd36da839894 is the first bad commit
commit 027140b7315efe3cd2e7b78fa608bd36da839894
Author: Maximilian Seidler <[email protected]>
Date:   Wed Sep 4 15:59:00 2024 +0000

    sessionLock: ensure sls focus in some edge cases (#7647)
    
    * input: return early in mouseMoveUnified when the session is locked
    
    * sessionLock: make make a commit an opportunity to focus session lock surfaces
    
    * compositor: allow resetting focus when session is locked
    
    * input: remove redundant PMONITOR checks
    
    PMONITOR is checked above
    
    * input: check isSessionLocked earlier in mouseMoveUnified
    
    A bit of reordering, so that we don't call some stuff that is irrelevant
    when the session is locked

 src/Compositor.cpp                  |  2 +-
 src/managers/SessionLockManager.cpp |  3 +++
 src/managers/input/InputManager.cpp | 56 ++++++++++++++++++++++++++++----------------------------
 3 files changed, 32 insertions(+), 29 deletions(-)

Description

dpms off when using hypridle does not work, monitors reactivate immediately

How to reproduce

Use hyprlock to run hyprctl dispatch dpms off after 10 seconds on lock screen.
Screen turns back on immediately with no input

Crash reports, logs, images, videos

No response

@dotaxis dotaxis added the bug Something isn't working label Sep 7, 2024
@dotaxis dotaxis changed the title dpms off not working properly dpms off not working properly on lock screen Sep 9, 2024
@PaideiaDilemma
Copy link
Contributor

I cannot reproduce this. How exactly are you running hyprctl dispatch dpms off.?

#7721 should resolve it I think.

@dotaxis
Copy link
Author

dotaxis commented Sep 9, 2024

With hypridle.

Will test your pr and let you know if it worked.

@dotaxis
Copy link
Author

dotaxis commented Sep 9, 2024

#7721 does fix :)

thanks!

@dotaxis dotaxis closed this as completed Sep 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants