Merge branch 'main' into feature/first_child

Author: willmcgugan

CHANGELOG.md

@@ -5,10 +5,23 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](http://keepachangelog.com/)
 and this project adheres to [Semantic Versioning](http://semver.org/).
 
+## Unreleased
+
 ### Fixed
 
 - Fixed `OptionList` causing excessive redrawing https://github.com/Textualize/textual/pull/5766
 - Added `:first-child` and `:last-child` pseudo classes https://github.com/Textualize/textual/pull/5776
+- Log messages could be written to stdout when there was no app, which could happen when using run_async or threads. Now they will be suppressed, unless the env var `TEXTUAL_DEBUG` is set https://github.com/Textualize/textual/pull/5782
+
+### Added
+
+- Added `toggle_class` parameter to reactives https://github.com/Textualize/textual/pull/5778
+- Added `compact` parameter and reactive to `Button`, `Input`, `ToggleButton`, `RadioSet`, `OptionList`, `TextArea` https://github.com/Textualize/textual/pull/5778
+- Added `HORIZONTAL_BREAKPOINTS` and `VERTICAL_BREAKPOINTS` to `App` and `Screen` https://github.com/Textualize/textual/pull/5779
+
+### Changed
+
+- `RadioSet` now has a default width of `1fr` https://github.com/Textualize/textual/pull/5778
 
 ## [3.1.1] - 2025-04-22
 

docs/guide/reactivity.md

@@ -65,7 +65,7 @@ You may want to add explicit type hints if the attribute type is a superset of t
 
 ## Smart refresh
 
-The first superpower we will look at is "smart refresh". When you modify a reactive attribute, Textual will make note of the fact that it has changed and refresh automatically.
+The first superpower we will look at is "smart refresh". When you modify a reactive attribute, Textual will make note of the fact that it has changed and refresh automatically by calling the widget's [`render()`][textual.widget.Widget.render] method to get updated content.
 
 !!! information
 

examples/breakpoints.py

@@ -0,0 +1,53 @@
+from textual.app import App, ComposeResult
+from textual.containers import Grid
+from textual.widgets import Footer, Markdown, Placeholder
+
+HELP = """\
+## Breakpoints
+
+A demonstration of how to make an app respond to the dimensions of the terminal.
+
+Try resizing the terminal, then have a look at the source to see how it works!
+"""
+
+
+class BreakpointApp(App):
+
+    # A breakpoint consists of a width and a class name to set
+    HORIZONTAL_BREAKPOINTS = [
+        (0, "-narrow"),
+        (40, "-normal"),
+        (80, "-wide"),
+        (120, "-very-wide"),
+    ]
+
+    CSS = """
+    Screen {        
+        Placeholder { padding: 2; }
+        Grid { grid-rows: auto; height: auto; }
+        # Change the styles according to the breakpoint classes
+        &.-narrow {
+            Grid { grid-size: 1; }
+        }
+        &.-normal {
+            Grid { grid-size: 2; }
+        }
+        &.-wide {
+            Grid { grid-size: 4; }
+        }
+        &.-very-wide {
+            Grid { grid-size: 6; }
+        }
+    }
+    """
+
+    def compose(self) -> ComposeResult:
+        yield Markdown(HELP)
+        with Grid():
+            for n in range(16):
+                yield Placeholder(f"Placeholder {n+1}")
+        yield Footer()
+
+
+if __name__ == "__main__":
+    BreakpointApp().run()

src/textual/__init__.py

@@ -85,8 +85,12 @@ def __call__(self, *args: object, **kwargs) -> None:
         try:
             app = active_app.get()
         except LookupError:
-            print_args = (*args, *[f"{key}={value!r}" for key, value in kwargs.items()])
-            print(*print_args)
+            if constants.DEBUG:
+                print_args = (
+                    *args,
+                    *[f"{key}={value!r}" for key, value in kwargs.items()],
+                )
+                print(*print_args)
             return
         if app.devtools is None or not app.devtools.is_connected:
             return
@@ -108,8 +112,12 @@ def __call__(self, *args: object, **kwargs) -> None:
             )
         except LoggerError:
             # If there is not active app, try printing
-            print_args = (*args, *[f"{key}={value!r}" for key, value in kwargs.items()])
-            print(*print_args)
+            if constants.DEBUG:
+                print_args = (
+                    *args,
+                    *[f"{key}={value!r}" for key, value in kwargs.items()],
+                )
+                print(*print_args)
 
     def verbosity(self, verbose: bool) -> Logger:
         """Get a new logger with selective verbosity.

src/textual/app.py

@@ -18,7 +18,7 @@
 import threading
 import uuid
 import warnings
-from asyncio import Task, create_task
+from asyncio import AbstractEventLoop, Task, create_task
 from concurrent.futures import Future
 from contextlib import (
     asynccontextmanager,
@@ -150,9 +150,6 @@
 if constants.DEBUG:
     warnings.simplefilter("always", ResourceWarning)
 
-# `asyncio.get_event_loop()` is deprecated since Python 3.10:
-_ASYNCIO_GET_EVENT_LOOP_IS_DEPRECATED = sys.version_info >= (3, 10, 0)
-
 ComposeResult = Iterable[Widget]
 RenderResult: TypeAlias = "RenderableType | Visual | SupportsVisual"
 """Result of Widget.render()"""
@@ -482,6 +479,31 @@ class MyApp(App[None]):
     SUSPENDED_SCREEN_CLASS: ClassVar[str] = ""
     """Class to apply to suspended screens, or empty string for no class."""
 
+    HORIZONTAL_BREAKPOINTS: ClassVar[list[tuple[int, str]]] | None = []
+    """List of horizontal breakpoints for responsive classes.
+
+    This allows for styles to be responsive to the dimensions of the terminal.
+    For instance, you might want to show less information, or fewer columns on a narrow displays -- or more information when the terminal is sized wider than usual.
+    
+    A breakpoint consists of a tuple containing the minimum width where the class should applied, and the name of the class to set.
+
+    Note that only one class name is set, and if you should avoid having more than one breakpoint set for the same size.
+
+    Example:
+        ```python
+        # Up to 80 cells wide, the app has the class "-normal"
+        # 80 - 119 cells wide, the app has the class "-wide"
+        # 120 cells or wider, the app has the class "-very-wide"
+        HORIZONTAL_BREAKPOINTS = [(0, "-normal"), (80, "-wide"), (120, "-very-wide")]
+        ```
+    
+    """
+    VERTICAL_BREAKPOINTS: ClassVar[list[tuple[int, str]]] | None = []
+    """List of vertical breakpoints for responsive classes.
+    
+    Contents are the same as [`HORIZONTAL_BREAKPOINTS`][textual.app.App.HORIZONTAL_BREAKPOINTS], but the integer is compared to the height, rather than the width.
+    """
+
     _PSEUDO_CLASSES: ClassVar[dict[str, Callable[[App[Any]], bool]]] = {
         "focus": lambda app: app.app_focus,
         "blur": lambda app: not app.app_focus,
@@ -2034,7 +2056,6 @@ async def run_async(
         from textual.pilot import Pilot
 
         app = self
-
         auto_pilot_task: Task | None = None
 
         if auto_pilot is None and constants.PRESS:
@@ -2067,27 +2088,29 @@ async def run_auto_pilot(
                     run_auto_pilot(auto_pilot, pilot), name=repr(pilot)
                 )
 
-        try:
-            app._loop = asyncio.get_running_loop()
-            app._thread_id = threading.get_ident()
-
-            await app._process_messages(
-                ready_callback=None if auto_pilot is None else app_ready,
-                headless=headless,
-                inline=inline,
-                inline_no_clear=inline_no_clear,
-                mouse=mouse,
-                terminal_size=size,
-            )
-        finally:
+        app._loop = asyncio.get_running_loop()
+        app._thread_id = threading.get_ident()
+        with app._context():
             try:
-                if auto_pilot_task is not None:
-                    await auto_pilot_task
+                await app._process_messages(
+                    ready_callback=None if auto_pilot is None else app_ready,
+                    headless=headless,
+                    inline=inline,
+                    inline_no_clear=inline_no_clear,
+                    mouse=mouse,
+                    terminal_size=size,
+                )
             finally:
                 try:
-                    await asyncio.shield(app._shutdown())
-                except asyncio.CancelledError:
-                    pass
+                    if auto_pilot_task is not None:
+                        await auto_pilot_task
+                finally:
+                    try:
+                        await asyncio.shield(app._shutdown())
+                    except asyncio.CancelledError:
+                        pass
+                app._loop = None
+                app._thread_id = 0
 
         return app.return_value
 
@@ -2100,6 +2123,7 @@ def run(
         mouse: bool = True,
         size: tuple[int, int] | None = None,
         auto_pilot: AutopilotCallbackType | None = None,
+        loop: AbstractEventLoop | None = None,
     ) -> ReturnType | None:
         """Run the app.
 
@@ -2111,36 +2135,24 @@ def run(
             size: Force terminal size to `(WIDTH, HEIGHT)`,
                 or None to auto-detect.
             auto_pilot: An auto pilot coroutine.
-
+            loop: Asyncio loop instance, or `None` to use default.
         Returns:
             App return value.
         """
 
         async def run_app() -> None:
             """Run the app."""
-            self._loop = asyncio.get_running_loop()
-            self._thread_id = threading.get_ident()
-            with self._context():
-                try:
-                    await self.run_async(
-                        headless=headless,
-                        inline=inline,
-                        inline_no_clear=inline_no_clear,
-                        mouse=mouse,
-                        size=size,
-                        auto_pilot=auto_pilot,
-                    )
-                finally:
-                    self._loop = None
-                    self._thread_id = 0
+            await self.run_async(
+                headless=headless,
+                inline=inline,
+                inline_no_clear=inline_no_clear,
+                mouse=mouse,
+                size=size,
+                auto_pilot=auto_pilot,
+            )
 
-        if _ASYNCIO_GET_EVENT_LOOP_IS_DEPRECATED:
-            # N.B. This doesn't work with Python<3.10, as we end up with 2 event loops:
-            asyncio.run(run_app())
-        else:
-            # However, this works with Python<3.10:
-            event_loop = asyncio.get_event_loop()
-            event_loop.run_until_complete(run_app())
+        event_loop = asyncio.get_event_loop() if loop is None else loop
+        event_loop.run_until_complete(run_app())
         return self.return_value
 
     async def _on_css_change(self) -> None:

src/textual/reactive.py

@@ -111,6 +111,7 @@ class Reactive(Generic[ReactiveType]):
         compute: Run compute methods when attribute is changed.
         recompose: Compose the widget again when the attribute changes.
         bindings: Refresh bindings when the reactive changes.
+        toggle_class: An optional TCSS classname(s) to toggle based on the truthiness of the value.
     """
 
     _reactives: ClassVar[dict[str, object]] = {}
@@ -126,6 +127,7 @@ def __init__(
         compute: bool = True,
         recompose: bool = False,
         bindings: bool = False,
+        toggle_class: str | None = None,
     ) -> None:
         self._default = default
         self._layout = layout
@@ -135,6 +137,7 @@ def __init__(
         self._run_compute = compute
         self._recompose = recompose
         self._bindings = bindings
+        self._toggle_class = toggle_class
         self._owner: Type[MessageTarget] | None = None
         self.name: str
 
@@ -175,6 +178,7 @@ def _initialize_reactive(self, obj: Reactable, name: str) -> None:
             name: Name of attribute.
         """
         _rich_traceback_omit = True
+
         internal_name = f"_reactive_{name}"
         if hasattr(obj, internal_name):
             # Attribute already has a value
@@ -308,6 +312,11 @@ def _set(self, obj: Reactable, value: ReactiveType, always: bool = False) -> Non
         public_validate_function = getattr(obj, f"validate_{name}", None)
         if callable(public_validate_function):
             value = public_validate_function(value)
+
+        # Toggle the classes using the value's truthiness
+        if (toggle_class := self._toggle_class) is not None:
+            obj.set_class(bool(value), *toggle_class.split())
+
         # If the value has changed, or this is the first time setting the value
         if always or self._always_update or current_value != value:
             # Store the internal value
@@ -407,6 +416,7 @@ class reactive(Reactive[ReactiveType]):
         always_update: Call watchers even when the new value equals the old value.
         recompose: Compose the widget again when the attribute changes.
         bindings: Refresh bindings when the reactive changes.
+        toggle_class: An optional TCSS classname(s) to toggle based on the truthiness of the value.
     """
 
     def __init__(
@@ -419,6 +429,7 @@ def __init__(
         always_update: bool = False,
         recompose: bool = False,
         bindings: bool = False,
+        toggle_class: str | None = None,
     ) -> None:
         super().__init__(
             default,
@@ -428,6 +439,7 @@ def __init__(
             always_update=always_update,
             recompose=recompose,
             bindings=bindings,
+            toggle_class=toggle_class,
         )
 
 
@@ -439,6 +451,7 @@ class var(Reactive[ReactiveType]):
         init: Call watchers on initialize (post mount).
         always_update: Call watchers even when the new value equals the old value.
         bindings: Refresh bindings when the reactive changes.
+        toggle_class: An optional TCSS classname(s) to toggle based on the truthiness of the value.
     """
 
     def __init__(
@@ -447,6 +460,7 @@ def __init__(
         init: bool = True,
         always_update: bool = False,
         bindings: bool = False,
+        toggle_class: str | None = None,
     ) -> None:
         super().__init__(
             default,
@@ -455,6 +469,7 @@ def __init__(
             init=init,
             always_update=always_update,
             bindings=bindings,
+            toggle_class=toggle_class,
         )