-
-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
picom: add windowType option #1163
Conversation
It was previously not possible to configure other windows types because the window key was already used.
}; | ||
}; | ||
default = { }; | ||
description = "Specific settings for this window type."; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This sounds very generic, specifically what does "this" refer to w.r.t window types?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The same holds for the descriptions above.
notify = rulesOption; | ||
combo = rulesOption; | ||
dnd = rulesOption; | ||
}; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
From what I can tell this will introduce something like 90 options, does picom make any guarantees that these options are safe w.r.t backward and forward compatibility? If not then this could be quite unpleasant to maintain. Would it be possible to do something more in line of RFC #42?
Thanks for the contribution! I've added a few comments. |
I'm inclined to close this PR without merging due to the large number of options and that nobody has been pushing particularly hard for this to go in. I will close for now, if anybody disagrees then feel free to make a comment and re-open. |
Description
It was previously not possible to configure other windows types through
extraConfig
because thewintypes
key was already used. Adding this option allows unlimited configuration.I kept
noDNDShadow
,noDockShadow
, andmenuOpacity
as shortcuts.Checklist
Code formatted with
./format
.Code tested through
nix-shell --pure tests -A run.all
. (no new failures)[ ] Test cases updated/added.Commit messages are formatted like