-
Notifications
You must be signed in to change notification settings - Fork 60
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
Border radius improvements #22
Comments
good point. it's pretty challenging to come up with a solid UI idea. its write-only? if the the selected borders all have the same value it gets displayed. if they differ, i first implemented that their median gets displayed, but switched it to setting the displayed value to zero. someone already suggested to just display every corners value in the representing selection filed below the slider. on elliptic border radius: does anyone really want that feature? expanding the UI to support them (e.g. slicing every selection box diagonally) would make the UI way more complex - is it worth it? |
I called it write-only because it feels that way. I didn't find a way to change one corner once I had set other corners. I agree that the UI should be too complicated, but an elliptic border radius, well, I use it from time to time. Perhaps it's not used widely, because few people know that it can be done. I think something like this: two sliders, and a checkbox to keep them synchronized (or not). By default they set both radii for all corners. Then a selector of some sort to select specific corners and set radii for them. |
changing on corner once the other corners are set should work by selecting said corner with the UI element below the slider. Did you find a bug there or is the handling complicated/unintuitive? the idea using two sliders and a checkbox sounds great! i would even go that far to hide the second slider unless the checkbox is checked. |
I guess it was just unintuitive. But I recon it's challenging to create a clear GUI for complicated settings like these. |
Border radius can get two improvements
The text was updated successfully, but these errors were encountered: