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

Parse border color props as gradient #5513

Merged
merged 1 commit into from
Apr 9, 2024
Merged

Conversation

SoSeDiK
Copy link
Contributor

@SoSeDiK SoSeDiK commented Apr 9, 2024

Describe your PR, what does it fix/add?

Border colors already support gradient, this lets setprop to parse gradient as well.

Is there anything you want to mention? (unchecked code, possible bugs, found problems, breaking compatibility, etc.)

Wiki states that -1 means unset, however, that wasn't actually the case before. This PR respects -1 and makes it act as unset rule.

Is it ready for merging, or does it need work?

Ready

@SoSeDiK
Copy link
Contributor Author

SoSeDiK commented Apr 9, 2024

I see that window/layer rules tend to use "unset". Could also use it in there (& eventually unify for all props?) instead of -1? 🤔 (considering -1 didn't work in the first place in this case, so not a breaking change)

@vaxerski
Copy link
Member

vaxerski commented Apr 9, 2024

no need I think

Copy link
Member

@vaxerski vaxerski left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@vaxerski vaxerski merged commit f2addfb into hyprwm:main Apr 9, 2024
9 checks passed
lisuke pushed a commit to lisuke/Hyprland that referenced this pull request Apr 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants