-
-
Notifications
You must be signed in to change notification settings - Fork 31.7k
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
[Table] Synchronize horizontal sticky header position with body #26159
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Make the sticky header look good when scrolling
Could you elaborate what you mean by "good"? The title is important for our changelog and "good" is an ambigious term.
sure. Made an update and welcome suggestions |
It makes sense, we have the same behavior on https://material-ui.com/components/data-grid/#commercial-version, the scroll position of the header and the body is synchronized horizontally. This style originates from the author that introduced the feature, it's the first time we rework it. |
@LiKang6688 It's a great first pull request on Material-UI 👌🏻. Thank you for working on it! |
HEAD:
https://next.material-ui.com/components/tables/#sticky-header
fixed-header.mov
PR:
https://deploy-preview-26159--material-ui.netlify.app/components/tables/#sticky-header
fixed-header.mov