Skip to content

Conversation

n3m0-22
Copy link
Contributor

@n3m0-22 n3m0-22 commented Sep 27, 2022

This changes the lower bound for the Active Border Radius to zero per user request #1522. This does not change the default value.

This changes the lower bound for the Active Border Radius to zero per user request #1522. This does not change the default value.
@n3m0-22 n3m0-22 marked this pull request as ready for review September 27, 2022 16:05
@n3m0-22 n3m0-22 requested review from a team September 27, 2022 16:05
13r0ck
13r0ck previously approved these changes Sep 27, 2022
@jacobgkau jacobgkau self-assigned this Sep 28, 2022
Copy link
Member

@jacobgkau jacobgkau left a comment

Choose a reason for hiding this comment

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

This allows me to change the option in the top-right menu below 5 (down to 0), but values below 5 do not take effect or propagate to the gsettings entry. Trying to set the value to below 5 via gsettings set also doesn't work. I think there's a remaining limit elsewhere in the code.

Screenshot from 2022-09-28 14-58-55

Copy link
Member

@jacobgkau jacobgkau left a comment

Choose a reason for hiding this comment

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

This is working now:
Screenshot from 2022-09-29 09-17-06
Does not affect the default.

@n3m0-22 n3m0-22 merged commit aaf86e4 into master_jammy Sep 29, 2022
@n3m0-22 n3m0-22 deleted the active-border-radius-min-zero branch September 29, 2022 15:36
@jacobgkau jacobgkau linked an issue Oct 2, 2022 that may be closed by this pull request
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.

[Feature Request] Allow border radius to be reduced to 0
4 participants