Conversation
26125ec to
8bee335
Compare
|
I have shortened the new |
|
Then "border radius bottom right" will be |
The border radius class will apply to all 4 corners, so there won't be a |
Yes, I guess I saw the implementation in tailwindcss. That came to my mind and wanted to ask. BTW the clarification with |
|
Not sure about the Not sure about that syntax either though. @mdo, what do you think? |
|
This PR does not include rename the border class name, but as @mdo commented above, the new name for border must also be considered. (This does not mean to include it in this PR.) e.g.
This also compatible with the spacing classes. |
|
@JensLuyten Could you change the class names to |
|
@ysds At that point, we might as well use |
|
@JensLuyten Could you please take a pass at updating this to reflect the latest comments? We're backpedaling on the abbreviations and are opting for |
Closes #29989
Updated the docs pages by searching for the
.roundedclasses and replacing them by the new.border-radiusclasses.See
https://deploy-preview-30048--twbs-bootstrap.netlify.com/docs/4.3/utilities/borders/#border-radius
https://deploy-preview-30048--twbs-bootstrap.netlify.com/docs/4.3/utilities/borders/#sizes