-
-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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
Adjust border-radius-large to fit with padding of modals #11536
Conversation
10px is decent I guess. But 15 is still too big. Why this sudden change? I mean, this doesn't really add anything ux speaking, this is obviously just a matter of visual ease. But I don't see what made you go for this? Did you saw it somewhere else (macOS you said?) |
Don't try to lure me with articles with 🐍 😁 |
@jancborchardt that article is talking about double border, not about rounded items withing rounded items, so I'm not sure it's relevant to this topic. And if it was, that would actually mean we would have to update corners for popovers as well - since those can also contain buttons (example: notifications). From what I see, new macOS doesn't follow this rule either - buttons are still way more rounded that P.S. Personally, the 10px radius is most eye-candy for me :) |
Signed-off-by: Jan-Christoph Borchardt <hey@jancborchardt.net>
2ceed4c
to
7931797
Compare
@pixelipo I know, but when the buttons are on the corner, it’s basically the same effect.
@skjnldsv Yup, normal rounded corners have visible "tips" at the start of the rounded corners – we can definitely look at that once … I guess CSS15 introduces that ability. ;D
@pixelipo alright – adjusted it to 10px. :) Please review. |
So the border-radius doesn’t have to be chosen just willy-nilly, but we have an anchor point and that are the buttons on the bottom (or generally elements inside):
This is how it looks in this pull request: 15px border-radius, to fit the 15px padding of .oc-dialog:
This is how it was after my pull request – yes that’s too big (but also accounted for the margin of the buttons):
This is 10px (would also happen with less), creating a bump → for reference see this admittedly quite old article, cause it’s an old issue: