inline improvements

Author: willmcgugan

src/textual/_compositor.py

@@ -148,8 +148,9 @@ def __rich_repr__(self) -> rich.repr.Result:
 class InlineUpdate(CompositorUpdate):
     """A renderable to write an inline update."""
 
-    def __init__(self, strips: list[Strip]) -> None:
+    def __init__(self, strips: list[Strip], clear: bool = False) -> None:
         self.strips = strips
+        self.clear = clear
 
     def __rich_console__(
         self, console: Console, options: ConsoleOptions
@@ -175,14 +176,15 @@ def render_segments(self, console: Console) -> str:
             append(strip.render(console))
             if not last:
                 append("\n")
-        append("\n\x1b[J")  # Clear down
+        if self.clear:
+            append("\n\x1b[J")  # Clear down
         if len(self.strips) > 1:
-            append(
-                f"\x1b[{len(self.strips)}A\r"
-            )  # Move cursor back to original position
+            back_lines = len(self.strips) if self.clear else len(self.strips) - 1
+            append(f"\x1b[{back_lines}A\r")  # Move cursor back to original position
         else:
             append("\r")
         append("\x1b[6n")  # Query new cursor position
+
         return "".join(sequences)
 
 
@@ -336,6 +338,9 @@ def __init__(self) -> None:
         # Mapping of line numbers on to lists of widget and regions
         self._layers_visible: list[list[tuple[Widget, Region, Region]]] | None = None
 
+        # Size of previous inline update
+        self._previous_inline_height: int | None = None
+
     @classmethod
     def _regions_to_spans(
         cls, regions: Iterable[Region]
@@ -1030,7 +1035,15 @@ def render_inline(
             A renderable.
         """
         visible_screen_stack.set([] if screen_stack is None else screen_stack)
-        return InlineUpdate(self.render_strips(size))
+        strips = self.render_strips(size)
+        clear = (
+            self._previous_inline_height is not None
+            and len(strips) < self._previous_inline_height
+        )
+        try:
+            return InlineUpdate(strips, clear=clear)
+        finally:
+            self._previous_inline_height = len(strips)
 
     def render_full_update(self) -> LayoutUpdate:
         """Render a full update.

src/textual/drivers/linux_inline_driver.py

@@ -207,6 +207,17 @@ def on_terminal_resize(signum, stack) -> None:
         self._key_thread = Thread(target=self._run_input_thread)
         send_size_event()
         self._key_thread.start()
+        self._request_terminal_sync_mode_support()
+        self._enable_bracketed_paste()
+
+    def _request_terminal_sync_mode_support(self) -> None:
+        """Writes an escape sequence to query the terminal support for the sync protocol."""
+        # Terminals should ignore this sequence if not supported.
+        # Apple terminal doesn't, and writes a single 'p' in to the terminal,
+        # so we will make a special case for Apple terminal (which doesn't support sync anyway).
+        if os.environ.get("TERM_PROGRAM", "") != "Apple_Terminal":
+            self.write("\033[?2026$p")
+            self.flush()
 
     @classmethod
     def _patch_lflag(cls, attrs: int) -> int:

src/textual/screen.py

@@ -893,7 +893,7 @@ def _get_inline_height(self, size: Size) -> int:
             inline_height = max(inline_height, int(min_height.resolve(size, size)))
         if max_height is not None:
             inline_height = min(inline_height, int(max_height.resolve(size, size)))
-        inline_height = min(self.app.size.height - 1, inline_height)
+        inline_height = min(self.app.size.height, inline_height)
         return inline_height
 
     def _screen_resized(self, size: Size):