auto table (#2123)

* auto table

* ws

* debug

CHANGELOG.md

@@ -9,6 +9,7 @@ and this project adheres to [Semantic Versioning](http://semver.org/).
 
 ### Changed
 
+- DataTable now has height: auto by default. https://github.com/Textualize/textual/issues/2117
 - Textual will now render strings within renderables (such as tables) as Console Markup by default. You can wrap your text with rich.Text() if you want the original behavior. https://github.com/Textualize/textual/issues/2120
 - Some widget methods now return `self` instead of `None` https://github.com/Textualize/textual/pull/2102:
   - `Widget`: `refresh`, `focus`, `reset_focus`
@@ -21,6 +22,7 @@ and this project adheres to [Semantic Versioning](http://semver.org/).
   - `TreeNode`: `expand`, `expand_all`, `collapse`, `collapse_all`, `toggle`, `toggle_all`
   - `Tree`: `clear`, `reset`
 
+
 ## [0.16.0] - 2023-03-22
 
 ### Added

src/textual/widgets/_data_table.py

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

tests/snapshot_tests/snapshot_apps/auto-table.py

@@ -69,6 +69,7 @@ class Status(LabeledBox):
 
     Status StatusTable {
         width: auto;
+        height: 100%;
         margin-top: 1;
         scrollbar-gutter: stable;
         overflow-x: hidden;

tests/test_data_table.py

@@ -646,6 +646,7 @@ async def test_hover_coordinate():
         table.add_column("ABC")
         table.add_row("123")
         table.add_row("456")
+        await pilot.pause()
         assert table.hover_coordinate == Coordinate(0, 0)
         await pilot.hover(DataTable, offset=Offset(2, 2))
         assert table.hover_coordinate == Coordinate(1, 0)