reactive stuff

Author: willmcgugan

src/textual/scrollbar.py

@@ -12,6 +12,7 @@
 
 log = logging.getLogger("rich")
 
+from . import events
 from .widget import Reactive, Widget
 
 

src/textual/widget.py

@@ -96,7 +96,8 @@ def __set__(self, obj: "Widget", value: ReactiveType) -> None:
                 if callable(update_function):
                     update_function(current_value, value)
 
-                obj.post_message_no_wait(events.Repaint(obj))
+                obj.require_repaint()
+                # obj.post_message_no_wait(events.Null(obj))
 
 
 @rich.repr.auto
@@ -145,6 +146,7 @@ def require_repaint(self) -> None:
         Actual repaint is done by parent on idle.
         """
         self._repaint_required = True
+        self.post_message_no_wait(events.Null(self))
 
     def check_repaint(self) -> bool:
         return True
@@ -214,9 +216,6 @@ async def on_idle(self, event: events.Idle) -> None:
             log.debug("REPAINTING")
             await self.repaint()
 
-    async def on_repaint(self, event: events.Repaint) -> None:
-        self.require_repaint()
-
 
 class Widget(WidgetBase):
     def __init__(self, name: str | None = None) -> None:

src/textual/widgets/placeholder.py

@@ -3,19 +3,18 @@
 from rich.console import Console, ConsoleOptions, RenderableType
 from rich.panel import Panel
 from rich.pretty import Pretty
-import rich.repr
+from rich.repr import RichReprResult
 
 from .. import events
 from ..widget import Reactive, Widget
 
 
-@rich.repr.auto
 class Placeholder(Widget, can_focus=True):
 
     has_focus: Reactive[bool] = Reactive(False)
     mouse_over: Reactive[bool] = Reactive(False)
 
-    def __rich_repr__(self) -> rich.repr.RichReprResult:
+    def __rich_repr__(self) -> RichReprResult:
         yield "name", self.name
         yield "has_focus", self.has_focus
         yield "mouse_over", self.mouse_over