Skip to content

Make sure we can tweak the custom-file-border-radius-after#28388

Closed
samdemaeyer wants to merge 2 commits intotwbs:masterfrom
samdemaeyer:add_custom_file_border_radius_after_variable
Closed

Make sure we can tweak the custom-file-border-radius-after#28388
samdemaeyer wants to merge 2 commits intotwbs:masterfrom
samdemaeyer:add_custom_file_border_radius_after_variable

Conversation

@samdemaeyer
Copy link

@samdemaeyer samdemaeyer commented Mar 1, 2019

Fixes #28389.
When the border width for input fields is more than 1px, the radius of the custom-file-border label after is not correct.
This PR introduces a way to customize that radius with a new variable: $custom-file-border-radius-after

Before:
screenshot 2019-03-01 at 09 20 58
screenshot 2019-03-01 at 09 26 33

After:
screenshot 2019-03-01 at 09 21 12
screenshot 2019-03-01 at 09 26 22

@samdemaeyer samdemaeyer requested a review from a team as a code owner March 1, 2019 11:45
@XhmikosR
Copy link
Member

XhmikosR commented Mar 1, 2019

@twbs/css-review: should this be backported in v4?

@alecpl
Copy link
Contributor

alecpl commented Mar 2, 2019

Maybe it would be better to remove border-radius on ::after element and add overflow:hidden to the label? Another ticket is also asking for overflow:hidden.

The ticket is #26933

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

custom-file-border-radius-after is not customizable

4 participants