-
-
Notifications
You must be signed in to change notification settings - Fork 79k
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
.invalid-feedback doesn't show for .input-group with an invalid control #29439
Comments
Currently, there is no reasonable way to solve this issue (and #25110) when using the constraint validation API for input groups. For v5, we should consider to remove support for the constraint validation API within input group, and support only CSS class-based (server-side) way. This change needs breaking change the HTML structure, so it may be difficult to fix this issue in v4. Or it may be possible to support a new usage while keeping the current usage as deprecated. |
Workarounds for V4: https://codepen.io/fellows3/pen/PoYgpxE?editors=1100 |
Thanks for showing some work arounds. Whats your logic for this?
I don't think it makes sense to remove validation features for some types of form elements/groups and not others. I would expect most people would expect consistent validations for all form elements/groups and variants of. |
The radio silence from the Bootstrap team on this issue is infuriating. This issue was first raised nearly two-and-a-half years ago. I’m sure this issue was raised when Bootstrap 4 was in alpha, and there have since been beta released and four 4.x releases where this fundamental issue has been ignored, yet work has been put into new components like spinners, toasts, and (yet another) icon set. @mdo, @fat, wider twbs/bootstrap team: can we please get an official response on this issue? I don’t understand why it’s just been ignored with no timeline or response given. |
just added d-block to the class of the feedback and it works like a charm.
|
This work as-is, but it doesn't round the corners properly. That's been the long outstanding issue we haven't been able to solve without breaking changes. See https://codepen.io/emdeoh/pen/PoqBdzN for an input group demo. Please note the markup difference here has the invalid feedback inside the input group, which is demonstrated in our site docs. @br3nt The start of your issue is about input groups, and then it's about custom files. It's unclear to me what the full issue is here, which likely explains our radio silence. As far as I know, the docs show what Bootstrap is currently capable of. For v5, we're reviewing some changes to these things, too. See #30170. |
#30180 shipped and addressed this as best as possible. Closing as we can't do much more as far as I can tell in v4. |
I'm reposting an outstanding bug that is documented in #23454 (closed). That issue has received ALOT of attention over a two year period but has not received a resolution.
Original issue:
Example:
Results in:
I would expect to see
can't be blank, is not a number
under the input-group.Summary of working hacks:
There are two working solutions (aka hacks) to get the
invalid-feedback
displaying forcustom-file
with aninput-group-append
.flex-wrap
class to thecustom-file
and adding thew-100
class to theinvalid-feedback
element.style="position: absolute; top: 3.0em"
to theinvalid-feedback
element.The downside to both of these is that
invalid-feedback
element no longer takes up space so elements below thecustom-file
don't accommodate by moving down to maintain expected spacing.Furthermore, The following solutions DO NOT work for
custom-file
:Adding the class
d-block
ord-flex
to theinvalid-feedback
element.Also, the
invalid-feedback
element already has the propertydisplay: block
.The text was updated successfully, but these errors were encountered: