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

Odd behavior when using EuiPopover in tables #959

Closed
cchaos opened this issue Jul 2, 2018 · 3 comments
Closed

Odd behavior when using EuiPopover in tables #959

cchaos opened this issue Jul 2, 2018 · 3 comments
Labels

Comments

@cchaos
Copy link
Contributor

cchaos commented Jul 2, 2018

This example just shows the sort dropdown, but the same behavior happens on any popover like multiple actions.

This example just starts in responsive layout for tables, but happens at wider widths too.

Link to gif if not showing inline

There are 3 things happening here:

  1. When clicking the popover button, the page auto-scrolls back to the top.
  2. The popover isn't getting the correct position until the browser's width is changed.
  3. When the browser's width is widened to get out of responsive table mode, which removes the sort dropdown, the popover stays open but doesn't have anything to position against anymore, so it's in the top left.

__

I consider this a blocker as this functionality can now break in current table implementations.

@cchaos
Copy link
Contributor Author

cchaos commented Jul 2, 2018

If you need another simpler doc page to see the behavior of no. 1, it happens to pagination as well.

@chandlerprall
Copy link
Contributor

Item 2 (correct position) is related to #955

@chandlerprall
Copy link
Contributor

2 is caused by 1 - the scroll jumps the window to the top so there is no placement which places the popover in the viewport.

Item 1 is caused by EuiContextMenuPanel's (used by EuiTableSortMobile) tabIndex="0".

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

No branches or pull requests

2 participants