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

[EuiFlyout] Popover and other positioned elements within should not scroll with the page #3456

Closed
Tracked by #4054
matchatype opened this issue May 11, 2020 · 5 comments

Comments

@matchatype
Copy link

Absolutely positioned elements do not seem to work correctly when they are wrapped in a flyout component. They scroll with the rest of the page.

Check the "More complicated flyout" on the official style guide for reproduction.

flyout

@snide
Copy link
Contributor

snide commented May 11, 2020

This is mostly capture by #1236

I'll close that one, since this one has a more general writeup.

@cchaos
Copy link
Contributor

cchaos commented Sep 20, 2020

The solution is to add repositionOnScroll={true} to the EuiPopover component. We've talked about a long-term goal that might be able to pass down a context to all EuiFlyout contents in order to supply this automatically. I'll leave this ticket open as a reminder of that goal.

@cchaos cchaos changed the title Positioned elements on flyout should not scroll with the page [EuiFlyout] Popover and other positioned elements within should not scroll with the page Sep 20, 2020
@github-actions
Copy link

👋 Hey there. This issue hasn't had any activity for 180 days. We'll automatically close it if that trend continues for another week. If you feel this issue is still valid and needs attention please let us know with a comment.

@cchaos
Copy link
Contributor

cchaos commented Mar 20, 2021

I've added this as a bullet point to #4439, so we don't need this ticket to stay open.

@cchaos cchaos closed this as completed Mar 20, 2021
@neeerajtk
Copy link

I had a similar issue when using EuiPopover inside EuiBasicTable. The popover moved along with the scroll. Solved it by adding onScroll event handler on the wrapping div of EuiBasicTable . onScroll={() => handleOpenedPopover(null)}

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

No branches or pull requests

4 participants