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

#25110 Missing border radius on input group with validation feedback #31518

Closed
wants to merge 2 commits into from

Conversation

dejakob
Copy link

@dejakob dejakob commented Aug 21, 2020

Issue reported

With new beta 3 and using the new input-group stuff, inputs inside input-groups with some validation feedback after them are missing the border-radius.

See: #25110

34426369-e70a858c-ec35-11e7-86f4-f178276e1f6e

Cause of issue

There is a selector in _input-group.scss that sets border-radius to 0 on all elements within an input-group that are not a dropdown or the last element:

.input-group {
  > :not(:last-child):not(.dropdown-toggle):not(.dropdown-menu)

Proposed solution

I added an extra pseudo-selector to check if there is no .invalid-feedback element coming after it, before overriding the border-radius to 0.

Screenshot 2020-08-21 at 14 11 48

@dejakob dejakob requested a review from a team as a code owner August 21, 2020 12:13
@ffoodd
Copy link
Member

ffoodd commented Sep 2, 2020

Thanks for submmitting a PR, and documenting it clearly!

However :has() has litterally no support at all… so your PR only invalidates the selector stack: look at .input-group-text, it get its right angles rounded.

As per comment, the current pattern does not allow to solve this: v5 would probably need to handle this by changing the HTML pattern, as stated in a comment too.

I'm closing this as a won't fix for now, but feel free to open another PR with a new pattern suggestion. This is one of our painpoint in v4, and we hope to find a way for v5 :)

@ffoodd ffoodd closed this Sep 2, 2020
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.

None yet

2 participants