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

cellStylesSticky zIndex doesn't work properly with Material UI Table with stickyHeader prop #4

Closed
sherbakovdev opened this issue Mar 17, 2020 · 5 comments

Comments

@sherbakovdev
Copy link

sherbakovdev commented Mar 17, 2020

The current zIndex interferes with MaterialUI Table with stickyHeader prop.

const cellStylesSticky = {
  position: 'sticky',
  zIndex: 2,
};

Please, increase the zIndex to 3 to solve the issue.

Screenshot 2020-03-17 at 11 41 41

Screenshot 2020-03-17 at 11 41 53

@sherbakovdev sherbakovdev changed the title cellStylesSticky zIndex doesn't work properly with Material UI Table with sticky Header cellStylesSticky zIndex doesn't work properly with Material UI Table with stickyHeader prop Mar 17, 2020
@GuillaumeJasmin
Copy link
Owner

@sherbakov1 thanks for your feedback :)
Please provide more informations (or better, a codesandbox example).
What is the current behavior ? Does Material UI override zIndex ?

@sherbakovdev
Copy link
Author

@GuillaumeJasmin Thanks for the quick reply! Please, check the images above :)

@GuillaumeJasmin
Copy link
Owner

GuillaumeJasmin commented Mar 17, 2020

Ok I get the issue !
I have release 1.1.0 yesterday with new css selector: https://github.com/GuillaumeJasmin/react-table-sticky/releases/tag/v1.1.0

You can temporary try:

[data-sticky-td] {
    position: sticky;
    z-index: 3 !important;
}

I will make a new release today with full CSS support and I think remove the inline zIndex

@sherbakovdev
Copy link
Author

Awesome! Great job! I will wait for the release.

@GuillaumeJasmin
Copy link
Owner

fixed on v1.1.2 with z-index: 3 . Sorry for the late publish.
In the next major version, I will remove hard coded inline style and use only CSS.
Thanks for your help :)

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

2 participants