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

[Bug] Popover locations for Sorting Controls overflowing #1240

Closed
2 of 4 tasks
thehig opened this issue Jul 12, 2018 · 4 comments
Closed
2 of 4 tasks

[Bug] Popover locations for Sorting Controls overflowing #1240

thehig opened this issue Jul 12, 2018 · 4 comments
Labels
Grid The DevExtreme Reactive Grid component question

Comments

@thehig
Copy link

thehig commented Jul 12, 2018

  • I have searched this repository's issues and believe that this is not a duplicate.

I'm using ...

  • React Grid
  • React Chart
  • Vue Grid

Current Behaviour

When creating a constrained table that performs its' own horizontal scrolling and utilizes the <TableHeaderRow showSortingControls />, shrinking the container beyond a minimum size will cause the entire page to create a horizontal scrollbar

Expected Behaviour

The usage of the Sorting Controls should not overflow the bounds of the Table

Steps to Reproduce (for Bugs)

Code Sample: https://codesandbox.io/s/rmpn3lw2zo

Using a Chrome plugin called Pesticide we can make the sorting control divs visible

image

And the corresponding HTML, showing that the popups are 'outside' the root of the application, which is as intended of course, but means i'm not sure how to resolve this

image

The popup anchors do update when resizing at larger sizes, but below some point, they stop tracking the corresponding UI components. However, performing mouseover on the Table Header (bringing up the 'sort' bubble) causes the popup to 'retrack' to the Header, and the scrollbar shrinks or goes away

@kvet
Copy link
Contributor

kvet commented Jul 13, 2018

Hi,
 
I've reproduced the issue and forwarded it to the original repository. Here is the link to the issue: mui/material-ui#12132
 
Now, as a workaround, I suggest you apply the following styles to the body element: overflow-x: hidden.

@thehig
Copy link
Author

thehig commented Jul 13, 2018

Thanks for forwarding the issue, much appreciated.

As for the workaround, I already tried that to no avail, but thanks for the suggestion

@thehig
Copy link
Author

thehig commented Jul 23, 2018

This has been resolved by the MUI team and the corresponding dependency update. Thanks! ❤️

@thehig thehig closed this as completed Jul 23, 2018
@dxrobot dxrobot added the Grid The DevExtreme Reactive Grid component label May 2, 2019
@dxrobot dxrobot unassigned kvet May 2, 2019
@lock
Copy link

lock bot commented Jun 15, 2019

This thread has been automatically locked since it is closed and there has not been any recent activity. Please open a new issue for related bugs or feature requests.

@lock lock bot locked as resolved and limited conversation to collaborators Jun 15, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Grid The DevExtreme Reactive Grid component question
Projects
None yet
Development

No branches or pull requests

3 participants