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

Adjust border-radius values and add some new utilities #35979

Merged
merged 2 commits into from Mar 11, 2022
Merged

Adjust border-radius values and add some new utilities #35979

merged 2 commits into from Mar 11, 2022

Conversation

mdo
Copy link
Member

@mdo mdo commented Mar 9, 2022

  • Updates global border-radius values for a more modern appearance
  • New .fw-semibold
  • New .rounded-4 and .rounded-5

Cherry-picked from #35736. /cc @XhmikosR

@mdo mdo requested a review from a team as a code owner March 9, 2022 17:04
@mdo mdo added this to In progress in v5.2.0 via automation Mar 9, 2022
v5.2.0 automation moved this from In progress to Reviewer approved Mar 11, 2022
mdo and others added 2 commits March 11, 2022 09:06
- Updates global border-radius values for a more modern appearance
- New .fw-semibold
- New .rounded-4 and .rounded-5
@mdo mdo merged commit ea7ff1e into main Mar 11, 2022
v5.2.0 automation moved this from Reviewer approved to Done Mar 11, 2022
@mdo mdo deleted the new-utils branch March 11, 2022 17:10
@fitcwill
Copy link

fitcwill commented Jul 23, 2022

@mdo signposted me to this PR when raising a point elsewhere (#36168) about something that related to this one...

Since upgrading to 5.2.0 to take advantage of some of the specific changes the border radius for a button and a card looked different.

In my case a btn-lg class is in use for the button as it's trying to help with the accessibility to have it a little larger... before the change the difference between the card and button radius was 0.05rem so although they look the same there is actually a slight difference.

Since the change the difference between the card and button is more prominent as it's different by 0.125rem. That difference is catching my eye more easily and not looking quite right. I couldn't see a class for altering the card radius to a large equivalent and imagine there isn't one.

I'm not really sure where else to continue discussing this as creating an issue to discuss it further didn't seem right either (or doesn't anymore at least). Happy to do that if you think best or wait for further guidance. Maybe a discussion is not worthwhile and it's just my tough luck and something I'll have to work around.

FWIW...

5.1.0 codepen - https://codepen.io/fitcwill/pen/ZExyvqM

5.2.0 codepen - https://codepen.io/fitcwill/pen/QWmgaZO

Ironically, the change I was starting to upgrade for makes the button look disabled when hovering over it and is similar to the disabled style itself. I actually feel more strongly about raising something for that!

@fitcwill
Copy link

I notice there was some discussion on this being seen as a breaking change by some... in this particular situation it does feel like it looks a bit broken and needs a work around / fix at least in my code now.

@mdo
Copy link
Member Author

mdo commented Jul 23, 2022

If you need a larger border-radius to hatch the large buttons, consider using a border radius utility class. Or you can update the card Sass or CSS variables.

@fitcwill
Copy link

Thanks @mdo. I'll look at doing exactly that when back on to that this week.

@fitcwill
Copy link

Regarding the style clash with the disabled style... any thoughts on me raising something for this?

@mdo
Copy link
Member Author

mdo commented Jul 26, 2022

I assume the disabled button thing you're talking about is that dark buttons appear slightly lighter on click/hover/focus. This was intentional as the previous states had too little of a change, so it was difficult to tell if they were clicked or focused. But if you have more questions on that, please open a new issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
No open projects
v5.2.0
  
Done
Development

Successfully merging this pull request may close these issues.

None yet

5 participants