Skip to content

Conversation

mdo
Copy link
Member

@mdo mdo commented Dec 10, 2020

Fixes #32356 and fixes #32188. Still might need some work, and I might be able to simplify things further.

Screen Shot 2020-12-10 at 9 24 43 AM

Screen Shot 2020-12-10 at 9 24 29 AM

@mdo
Copy link
Member Author

mdo commented Dec 10, 2020

Second commit here makes some slightly bigger changes to simplify the padding. Unsure if it's a possible breaking change though, so would love feedback from @twbs/css-review.

Screen Shot 2020-12-10 at 9 31 46 AM

Screen Shot 2020-12-10 at 9 31 36 AM

@XhmikosR XhmikosR requested a review from a team December 14, 2020 09:31
@MartijnCuppens
Copy link
Member

The complex calc() functions were added to prevent breaking changes in v4.3, but your simpler approach makes more sense to me. I'm ok with simplifying it a little at this point.

@mdo mdo marked this pull request as ready for review December 15, 2020 06:57
@XhmikosR XhmikosR merged commit a48a29a into main Dec 28, 2020
@XhmikosR XhmikosR deleted the form-select-padding branch December 28, 2020 06:27
sijusamson pushed a commit to sijusamson/bootstrap that referenced this pull request Jan 4, 2021
* Tweak .form-select padding

* Simplify the padding entirely

Co-authored-by: XhmikosR <[email protected]>
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.

[v5] .form-select arrow not aligned perfectly Little more right-margin for "info icon" in <select> when class is "is-invalid"
3 participants