-
-
Notifications
You must be signed in to change notification settings - Fork 78.8k
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
Fixes input group btn height bug #23810
Conversation
fe73370
to
7994ff5
Compare
7994ff5
to
73593b1
Compare
I hope that fixes the issue also for a (content-editable) div inside of the input-group. |
@alecpl I'd have to test it, but I think that as long as you add the class |
Here is a codepen to demo the fix: https://codepen.io/andresgalante/pen/QMXRwa |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good—curious if this also fixes #22690?
@mdo, It would be nice to have it in beta2. I also tested the change and works for me. Thanks. |
Nice! |
This is a follow up from this PR #23806.
The elements within input-group now stretches to avoid bugs like the one described on #23745, it now looks like this:
@mattcheah Please feel free to take this PR as an example to fix yours if you want. If you do please close this PR
This is fixing the problems introduced on this commit: 02ae73f
Fixes #23745