Skip to content

Menu form is broken in input group #25278

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

Closed
ysds opened this issue Jan 12, 2018 · 7 comments
Closed

Menu form is broken in input group #25278

ysds opened this issue Jan 12, 2018 · 7 comments

Comments

@ysds
Copy link
Contributor

ysds commented Jan 12, 2018

When using a menu form in a input group, the form controls in the dropdown will be broken.

image

Demo: https://jsfiddle.net/tdf22y9t/

The usability of such UI may be questionable, but does bootstrap support this usage? If support it, hopefully I can work on solving this problem by adding a selector >.

@mdo
Copy link
Member

mdo commented Jan 12, 2018

Yes, we should support it.

@ysds
Copy link
Contributor Author

ysds commented Jan 12, 2018

OK 👍 I created a PR.

@slidenerd
Copy link

Yup i double side this. Also try adding 2 input groups with a dropdown form in each, one of them ends up broken

@ysds
Copy link
Contributor Author

ysds commented Jan 15, 2018

Could you make a pen?

@slidenerd
Copy link

slidenerd commented Jan 15, 2018

Sure thank you for the quick response @ysds HERE is the pen Once you expand the dropdown menu and inspect element, you ll notice that only one of them has the form inside while the other one doesnt, any suggestions are super appreciated

What I am trying to build in that snippet is an input group with a dropdown button that has the ability to search items inside. I am adding a form as one of the children with a search box, a div inside which I will populate dropdown-items from ajax call or json and an item to show when there is nothing to display after filtering

@ysds
Copy link
Contributor Author

ysds commented Jan 15, 2018

The bootstrap CSS on your pen is not latest v4-dev. Also, I'm not sure your problem related this issue.

@slidenerd
Copy link

Thank you for pointing that out. I updated it to the latest bootstrap version 4 beta 3 , the issue still exists :)

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

No branches or pull requests

4 participants