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

TableBody does NOT vertically scroll #12014

Closed
2 tasks done
etayluz opened this issue Jun 29, 2018 · 5 comments
Closed
2 tasks done

TableBody does NOT vertically scroll #12014

etayluz opened this issue Jun 29, 2018 · 5 comments
Labels
duplicate This issue or pull request already exists

Comments

@etayluz
Copy link

etayluz commented Jun 29, 2018

  • This is a v1.x issue (v0.x is no longer maintained).
  • I have searched the issues of this repository and believe that this is not a duplicate.

Expected Behavior

TableBody should scroll vertically while TableHead is frozen in place.:
https://stackoverflow.com/questions/51068118/material-ui-give-tablebody-a-max-height-and-make-it-vertically-scrollable/51107369#51107369

Current Behavior

TableBody and TableHead scroll together (TableHead should be frozen):
https://stackoverflow.com/questions/51068118/material-ui-give-tablebody-a-max-height-and-make-it-vertically-scrollable/51107369#51107369

Steps to Reproduce (for bugs)

https://codesandbox.io/s/y03vmkkkqj

Context

This is such a basic feature for any table, how does Material UI NOT support this? That's ridiculous
Paging is supported, but who wants to page? It's such an annoying/slow user experience.
Infinite scrolling rocks!

Your Environment

Tech Version
Material-UI v1.0.0
@oliviertassinari oliviertassinari added the duplicate This issue or pull request already exists label Jun 29, 2018
@mbrookes
Copy link
Member

This is such a basic feature for any table, how does Material UI NOT support this? That's ridiculous

@etayluz If Material-UI is so far from your needs that it provokes this sort of reaction, you might want to explore alternatives. Being the world's favourite React UI library doesn't make it right for everyone, and you're under no obligation to use it.

OTOH, if you feel you can contribute a meaningful improvement, we're all ears.

@oliviertassinari
Copy link
Member

oliviertassinari commented Jun 29, 2018

@etayluz It's a duplicate of #6625.

@jaredkirkley
Copy link

In case anyone else is looking for this later, I achieved an effect close to what I was looking for with stickyHeader on the Table component with overflow: auto; on the parent container. I was also looking for a sticky footer which I was able to achieve using the same approach as the sticky header:

    .MuiTablePagination-root {
      position: sticky;
      bottom: 0;
      background-color: #FAFAFA;
    }

@nichita-pasecinic
Copy link

Hi, @oliviertassinari how this solution (#6625 (comment)) solves this issue ?
Just having a sticky position header does not solves the scrollbar issue on TableBody
Could you please help here ?

@Ashish-21
Copy link

Hi, @oliviertassinari how this solution (#6625 (comment)) solves this issue ? Just having a sticky position header does not solves the scrollbar issue on TableBody Could you please help here ?

Do you got any solution for this?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
duplicate This issue or pull request already exists
Projects
None yet
Development

No branches or pull requests

6 participants