-
-
Notifications
You must be signed in to change notification settings - Fork 79.1k
dropdown-menu-right works only inside navbars #23553
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
Comments
Bug reports must include a live demo of the problem. Per our contributing guidelines, please create a reduced test case via JS Bin or CodePen and report back with your link, Bootstrap version, and specific browser and OS details. |
I used this CDNs: https://cdnjs.cloudflare.com/ajax/libs/twitter-bootstrap/4.0.0-beta/css/bootstrap.min.css Here is the live example: https://codepen.io/Neimeg/pen/XaZVyd |
hummm ...the dropdown-menu-right class seems not exist alone in the css ..? so if you add this in the code pen css it works ... |
Dropdown now depends on Popper.js so the positioning is done by Popper.js that's why you didn't find the CSS. |
it doesnt work with too ... |
so why we have those all classes (for each grid step) are they still usefull ?? |
We made the choice to disable Popper.js for Dropdown in navbar because currently we don't have the need to use Popper.js |
cool
is true |
Yep that's true I'll reopen to let @mdo answer if we should re introduce this CSS class in this particular use case |
Just my thoughts. Currently dropdown in BS i a little bit chaotic. There is With current 'default' BS navbar implementation people can't use popper.js to position dropdown but there are cases when it is very helpful to use popper.js (kebab like menus, notification tips, etc) I think the best solution is to add other placement from popper.js (top-left, bottom-left, top-right, bottom-right) and introduce placement Currently .navbar have a lot styles for dropdown so why not move this .dropdown-* classes to dropdown.scss? There is also PR that adds another dropdown class to .navbar #23520 Of course docs should mention that in navbar dropdown should use 'static' placement. |
Bumping this as the bootstrap docs don't indicate that |
It's written here: https://getbootstrap.com/docs/4.1/components/navbar/#supported-content BTW there is no decision made here, so Dropdowns depend on Popper.js so if you want align your dropdown to the right, include Popper.js 👍 |
dropdown-menu-right doesn't work with pure CSS if you put it outside navbars.
The text was updated successfully, but these errors were encountered: