`DataTable` scrolling issues with `max-height: 100vh` about textual

[github-actions]

We found the following entry in the FAQ which you may find helpful:

• Why doesn't the DataTable scroll programmatically?

Feel free to close this issue if you found an answer in the FAQ. Otherwise, please give us a little time to review.

This is an automated reply, generated by FAQtory

[willmcgugan]

This is the expected behaviour, given the DataTable is larger than its container and the container doesn't scroll.

Having a table of height 100% caused a lot of confusion with auto height containers. Because the container is saying "I'll be as tall as the children" and the child is saying "Ill be as high as my parent.". Setting the default height to "100vh" fixes that, but it does mean that it may not fit inside a single screen.

It's simple enough to set the height to 100% or 1fr though.

[TomJGooding]

Wasn't this issue with auto sized containers addressed in a later PR?

If I change the CSS in the example to the below, I don't see the issue you're describing on v0.52.1?

    CSS = """
    Horizontal {
        border: solid red;
        height: auto;
    }

    DataTable {
        border: solid pink;
        max-height: 100%;
    }
    """

[TomJGooding]

After looking back through the issues and PRs, it looks like this issue #2975 was fixed in #3814.

Would you consider a PR to change the default max-height of the DataTable back to 100%?