-
-
Notifications
You must be signed in to change notification settings - Fork 1k
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
windowrules: add fullscreenstate field #7466
Conversation
ca3dd19
to
d140bf5
Compare
why not make "any" a |
d140bf5
to
923b095
Compare
923b095
to
a4833b6
Compare
Made any a |
with this we can nuke fullscreen in a future mr |
@vaxerski can we get better documentation around what "internal" and "client" refers to? |
MRs welcome. Internal means what hyprland does. Client means what the client receives |
I'm happy to open a PR(MR?) if you can help me understand the implications. Specifically I have tried every combination of settings of If this isn't the appropriate place to discuss this let me know how I can reach out. |
F11 is controlled by brave, so it will control the client fullscreen mode |
@MightyPlaza This worked perfectly. My problem is I was trying to apply it as a window rule. I'll open a PR to update the documentation soon. Thanks! |
@MightyPlaza @vaxerski thanks kings 👑 👉🏻 hyprwm/hyprland-wiki#892 |
Describe your PR, what does it fix/add?
Add
fullscreenstate
field for window rules v2.fullscreenstate: [internal] [client]
matches windows with specific fullscreen state.For example
fullscreenstate:0 1
matches windows that have internal state of 0 and client state of 1.fullscreenstate:* 2
matches windows that have client state of 2 (internal state can be anything).Is there anything you want to mention? (unchecked code, possible bugs, found problems, breaking compatibility, etc.)
Accompanying wiki MR: hyprwm/hyprland-wiki#757
Is it ready for merging, or does it need work?
Ready for merging.