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

Idle inhibitor not working with layer surface #5878

Closed
sungyoonc opened this issue May 4, 2024 · 7 comments
Closed

Idle inhibitor not working with layer surface #5878

sungyoonc opened this issue May 4, 2024 · 7 comments
Labels
bug Something isn't working

Comments

@sungyoonc
Copy link
Contributor

Hyprland Version

System/Version info
Hyprland, built from branch main at commit a3309b51a254f142c535022615be6f76eb23fa9b  (shadow: fix small pixel gaps between border).
Date: Sun May 5 04:30:03 2024
Tag: v0.40.0-1-ga3309b51, commits: 4607

flags: (if any)

Bug or Regression?

Bug

Description

When idle inhibitor is associated with a layer surface (e.g. waybar) or is not associated to any window, idle inhibitor does not work.

How to reproduce

Activate idle inhibitor with waybar

Crash reports, logs, images, videos

[LOG] Registered signal for owner 5f0a3147c260: 5f0a31462808 -> 5f0a3147c2c0 (owner: CIdleInhibitorResource)
[LOG] New idle inhibitor registered for surface 5f0a314624f0
[WARN] Inhibitor is for no window?
[LOG] Registered signal for owner 5f0a314fec50: 5f0a31463bd8 -> 5f0a314fecb0 (owner: CIdleInhibitorResource)
[LOG] New idle inhibitor registered for surface 5f0a314638c0
[WARN] Inhibitor is for no window?
@sungyoonc sungyoonc added the bug Something isn't working label May 4, 2024
@vaxerski
Copy link
Member

vaxerski commented May 4, 2024

oopsie daisy

@vaxerski
Copy link
Member

vaxerski commented May 4, 2024

patch.txt
does this work?

@vaxerski
Copy link
Member

vaxerski commented May 4, 2024

made some minor changes and it works.

@sungyoonc
Copy link
Contributor Author

Yup, it's working now thanks

@zakk4223
Copy link
Contributor

zakk4223 commented May 5, 2024

Pretty sure this broke command line inhibitors like wljoywake and wayland-idleinhibitor

@sungyoonc
Copy link
Contributor Author

Pretty sure this broke command line inhibitors like wljoywake and wayland-idleinhibitor

@zakk4223 Can you test this? #5882

@zakk4223
Copy link
Contributor

zakk4223 commented May 5, 2024

That fixes it

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

No branches or pull requests

3 participants