From 8c3bc4135ec58ccac19ead529ea563d1f2583665 Mon Sep 17 00:00:00 2001 From: Darren Burns Date: Tue, 25 Jan 2022 14:20:32 +0000 Subject: [PATCH] Add missing docstring, use explicit kwargs, remove id=_root --- src/textual/app.py | 2 +- src/textual/css/_style_properties.py | 14 +++++++------- src/textual/css/stylesheet.py | 3 ++- 3 files changed, 10 insertions(+), 9 deletions(-) diff --git a/src/textual/app.py b/src/textual/app.py index 8708bf5ab..96ca521e4 100644 --- a/src/textual/app.py +++ b/src/textual/app.py @@ -503,7 +503,7 @@ class App(DOMNode): # Handle input events that haven't been forwarded # If the event has been forwarded it may have bubbled up back to the App if isinstance(event, events.Mount): - view = View(id="_root") + view = View() self.register(self, view) await self.push_view(view) await super().on_event(event) diff --git a/src/textual/css/_style_properties.py b/src/textual/css/_style_properties.py index 12c9075f7..ff673fa10 100644 --- a/src/textual/css/_style_properties.py +++ b/src/textual/css/_style_properties.py @@ -257,7 +257,7 @@ class SpacingProperty: return getattr(obj, self._internal_name) or NULL_SPACING def __set__(self, obj: Styles, spacing: SpacingDimensions) -> Spacing: - obj.refresh(True) + obj.refresh(layout=True) spacing = Spacing.unpack(spacing) setattr(obj, self._internal_name, spacing) return spacing @@ -272,7 +272,7 @@ class DocksProperty: def __set__( self, obj: Styles, docks: Iterable[DockGroup] | None ) -> Iterable[DockGroup] | None: - obj.refresh(True) + obj.refresh(layout=True) if docks is None: obj._rule_docks = None else: @@ -285,7 +285,7 @@ class DockProperty: return obj._rule_dock or "" def __set__(self, obj: Styles, spacing: str | None) -> str | None: - obj.refresh(True) + obj.refresh(layout=True) obj._rule_dock = spacing return spacing @@ -315,7 +315,7 @@ class LayoutProperty: """ from ..layouts.factory import get_layout, Layout # Prevents circular import - obj.refresh(True) + obj.refresh(layout=True) if isinstance(layout, Layout): new_layout = layout else: @@ -335,7 +335,7 @@ class OffsetProperty: def __set__( self, obj: Styles, offset: tuple[int | str, int | str] | ScalarOffset ) -> tuple[int | str, int | str] | ScalarOffset: - obj.refresh(True) + obj.refresh(layout=True) if isinstance(offset, ScalarOffset): setattr(obj, self._internal_name, offset) return offset @@ -403,7 +403,7 @@ class NameProperty: return getattr(obj, self._internal_name) or "" def __set__(self, obj: Styles, name: str | None) -> str | None: - obj.refresh(True) + obj.refresh(layout=True) if not isinstance(name, str): raise StyleTypeError(f"{self._name} must be a str") setattr(obj, self._internal_name, name) @@ -423,7 +423,7 @@ class NameListProperty: def __set__( self, obj: Styles, names: str | tuple[str] | None = None ) -> str | tuple[str] | None: - obj.refresh(True) + obj.refresh(layout=True) names_value: tuple[str, ...] | None = None if isinstance(names, str): names_value = tuple(name.strip().lower() for name in names.split(" ")) diff --git a/src/textual/css/stylesheet.py b/src/textual/css/stylesheet.py index 616bcf1b2..8c6075011 100644 --- a/src/textual/css/stylesheet.py +++ b/src/textual/css/stylesheet.py @@ -108,7 +108,8 @@ class Stylesheet: yield selector_set.specificity def apply(self, node: DOMNode) -> None: - """ + """Apply the stylesheet to a DOM node. + Args: node (DOMNode): The ``DOMNode`` to apply the stylesheet to. Applies the styles defined in this ``Stylesheet`` to the node.