fix #569: bump fastdatatable to escape cell contents; improve tooltip…

… formatting and styling

CHANGELOG.md

@@ -11,10 +11,12 @@ All notable changes to this project will be documented in this file.
 ### Changes
 
 - Harlequin's Footer has been re-designed. In the footer, `CTRL+` key presses are now represented by a carat, `^`. For example, instead of `CTRL+Q Quit` the footer now reads `^q Quit`.
+- The tooltip for overflowing data cells has been improved to better format the data contained in the cell.
 
 ### Bug Fixes
 
 - Fixed a bug where the main panel would resize while the code editor was being mounted at app start-up.
+- Fixed a bug that could cause a crash if data table cell contents contained a string that could be interpreted as bad Rich Markup ([#569](https://github.com/tconbeer/harlequin/issues/569) - thank you [@cmdkev](https://github.com/cmdkev)!)
 
 ## [1.22.2] - 2024-07-09
 

pyproject.toml

@@ -23,7 +23,7 @@ python = ">=3.8.1,<4.0.0"
 
 # textual and component libraries
 textual = "==0.72.0"
-textual-fastdatatable = "==0.7.1"
+textual-fastdatatable = "==0.8.0"
 textual-textarea = "==0.14.0"
 
 # click

src/harlequin/app.tcss

@@ -148,6 +148,8 @@ Tooltip {
     border: round $border-color-focus;
     background: $background;
     padding: 0 1;
+    max-height: 95vh;
+    max-width: 40%;
 }
 
 RunQueryBar Button {

tests/functional_tests/test_app.py

@@ -74,12 +74,12 @@ async def test_select_1(
         "select 1 where false",
         "select 1 union all select 'hi'",
         "select 'hi' union all select 1",
+        """select '[/foo]' as "[/bar]" """.strip(),
     ],
 )
 async def test_queries_do_not_crash_all_adapters(
     app_all_adapters: Harlequin,
     query: str,
-    app_snapshot: Callable[..., Awaitable[bool]],
 ) -> None:
     app = app_all_adapters
     messages: list[Message] = []