added selection_updated method

Author: willmcgugan

src/textual/screen.py

@@ -41,6 +41,7 @@
 )
 from textual._spatial_map import SpatialMap
 from textual._types import CallbackType
+from textual.actions import SkipAction
 from textual.await_complete import AwaitComplete
 from textual.binding import ActiveBinding, Binding, BindingsMap
 from textual.css.match import match
@@ -342,7 +343,7 @@ async def _watch_selections(
         selections: dict[Widget, Selection],
     ):
         for widget in old_selections.keys() | selections.keys():
-            widget.refresh()
+            widget.selection_updated(selections.get(widget, None))
 
     def refresh_bindings(self) -> None:
         """Call to request a refresh of bindings."""
@@ -868,9 +869,10 @@ def get_selected_text(self) -> str | None:
     def action_copy_text(self) -> None:
         """Copy selected text to clipboard."""
         selection = self.get_selected_text()
-        if selection is not None:
-            self.app.copy_to_clipboard(selection)
-            self.notify(selection)
+        if selection is None:
+            # No text selected
+            raise SkipAction()
+        self.app.copy_to_clipboard(selection)
 
     def action_maximize(self) -> None:
         """Action to maximize the currently focused widget."""

src/textual/widget.py

@@ -3832,6 +3832,14 @@ def get_selection(self, selection: Selection) -> tuple[str, str] | None:
             return None
         return selection.extract(text), "\n"
 
+    def selection_updated(self, selection: Selection | None) -> None:
+        """Called when the selection is updated.
+
+        Args:
+            selection: Selection information or `None` if no selection.
+        """
+        self.refresh()
+
     def _render_content(self) -> None:
         """Render all lines."""
         width, height = self.size

src/textual/widgets/_log.py

@@ -272,6 +272,10 @@ def get_selection(self, selection: Selection) -> tuple[str, str] | None:
         text = "\n".join(self._lines)
         return selection.extract(text), "\n"
 
+    def selection_updated(self, selection: Selection | None) -> None:
+        self._render_line_cache.clear()
+        self.refresh()
+
     def render_line(self, y: int) -> Strip:
         """Render a line of content.
 
@@ -322,7 +326,9 @@ def _render_line_strip(self, y: int, rich_style: Style) -> Strip:
 
         _line = self._process_line(self._lines[y])
 
-        line_text = Text(_line, style=rich_style, no_wrap=True)
+        line_text = Text(_line, no_wrap=True)
+        line_text.stylize(rich_style)
+
         if self.highlight:
             line_text = self.highlighter(line_text)
         if selection is not None: