-
-
Notifications
You must be signed in to change notification settings - Fork 591
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
Feature request: add predefined targets for sticky and hidden #512
Comments
I think the problem can be solved by performing the matching against all property values if no index is specified, i.e.
should fit your expectation of setting opacity to 0 for a window that has The presets, like |
I agree with the approach of switching behavior to use the if-no-index-is-given-assume-anywhere as this is more general than my suggestion of special cases for hidden/sticky. Though I have to say, that reading them is something you have to get used to. Introducing a new predicate like the suggested
Additional checks (I cannot come up with examples at the moment) could be added to their respective functions if needed. Judging from the history of |
Sadly the current matching implementation doesn't really lend itself to this kind of matching with custom functions (for |
I've made the
You can test it here. |
I've tested my settings with the new key predicates for |
#550 adds an updated syntax to match against any value in an atom-array.
We have decided against adding the new |
I've adjusted my configuration according to the new implementation for testing and it works as intended. I guess, the ticket can be closed now (as it's already merged to |
Thanks for the feedback. I'm closing this now. |
For matching windows for e.g.
opacity-rule
orshadow-exclude
it might be handy to provide shortcuts forhidden
andsticky
(like the undocumentedfullscreen
) to avoid having to state several entries for each (as it's not known upfront on which entry within_NET_WM_STATE
the entry in question ends up).E.g. the example from ArchWiki
could be written as
instead.
A quickly made implementation can be found on my fork.
The text was updated successfully, but these errors were encountered: