-
-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
Dropdown boundary="window" fails within responsive table header #3563
Comments
Are you using the latest version of BootstrapVue? It is working in this fiddle: https://jsfiddle.net/0r4acg7o/ Column "A" is using the default boundary ( |
Ah, I placed the dropdowns in the |
Hi tmorehouse. Thanks for the quick response and the fiddle. I am indeed using the latest version. The issue appears to be specific to the table header. I forked your fiddle to show this: https://jsfiddle.net/04xb891t/3/ Any ideas? |
We have style Turning off that style, the dropdown works with boundary="window" |
|
Dropdown appears to require Would adding Thanks! |
Adding class |
I'm not sure if we have a simple fix for this at the moment, due to the requirements of the sorting icons needing position relative (for absolute positioning) |
The other option for us would be to make the sort control icons inline-blocks rather than absolute positioning |
PR #3568 should now accommodate for this by removing the You can try it out at https://deploy-preview-3568--bootstrap-vue.netlify.com/ (and try in the playground) |
…3568) Switch from absolute positioning to `display: inline-block` + `float: right` + `::before` (instead of `::after`) for sort icons. Removes need for `position: relative;` on table header/footer cells which can interfere with dropdowns and tooltips/popovers in headers of responsive tables (note `boundary` option needs to be set on dropdown, tooltip, popover, to allow to break out of responsive table container)
Version 2.0.0-rc.25 has been released. |
I have a responsive table and a dropdown in one of its headers. If the table content is shorter than the dropdown, the dropdown is trapped inside the scrollbox of the table. I figured I could fix this easily with
boundary="window"
as I have done on other responsive tables to fix tooltips. But this does not seem to do anything. I tried to reproduce the issue using native Bootstrap and the issue does not reproduce (i.e. addingdata-boundary="window"
does indeed work using only Bootstrap).Link to reproduce: https://codesandbox.io/s/goofy-joliot-v9b8f
Since this has
boundary="window"
, I would expect the dropdown to breakout of the.table-responsive
div.Please let me know if you need any further information.
The text was updated successfully, but these errors were encountered: