fix(datatable): change max-height back to 100%

The `max-height` of the `DataTable` was changed to 100vh rather than
100% in #3566, because at the time this caused issues with auto height
containers, as described in #2975.

However this issue was later fixed in #3814. This PR changes the
`max-height` of the `DataTable` back to 100%, as 100vh will actually
break common layouts and seems no longer required to workaround this
auto height issue.

Fixes #4286.

Author: TomJGooding

CHANGELOG.md

@@ -34,6 +34,7 @@ and this project adheres to [Semantic Versioning](http://semver.org/).
 - Fixed a style leak from `TabbedContent` https://github.com/Textualize/textual/issues/4232
 - Fixed active hidden scrollbars not releasing the mouse https://github.com/Textualize/textual/issues/4274
 - Fixed the mouse not being released when hiding a `TextArea` while mouse selection is happening https://github.com/Textualize/textual/issues/4292
+- Fixed `DataTable` scrolling issues by changing `max-height` back to 100% https://github.com/Textualize/textual/issues/4286
 
 ### Changed
 

src/textual/widgets/_data_table.py

@@ -266,7 +266,7 @@ class DataTable(ScrollView, Generic[CellType], can_focus=True):
         background: $surface ;
         color: $text;
         height: auto;
-        max-height: 100vh;
+        max-height: 100%;
     }
     DataTable > .datatable--header {
         text-style: bold;