urwid 2.6.15__py3-none-any.whl → 3.0.5__py3-none-any.whl

urwid/widget/grid_flow.py

@@ -144,8 +144,9 @@ class GridFlow(WidgetWrap[Pile], WidgetContainerMixin, WidgetContainerListConten
         """
         warnings.warn(
             "only for backwards compatibility."
-            "You should use the new standard container property `contents` to modify GridFlow",
-            PendingDeprecationWarning,
+            "You should use the new standard container property `contents` to modify GridFlow."
+            "API will be removed in version 5.0.",
+            DeprecationWarning,
             stacklevel=2,
         )
         ml = MonitoredList(w for w, t in self.contents)
@@ -160,33 +161,16 @@ class GridFlow(WidgetWrap[Pile], WidgetContainerMixin, WidgetContainerListConten
     def cells(self, widgets: Sequence[Widget]):
         warnings.warn(
             "only for backwards compatibility."
-            "You should use the new standard container property `contents` to modify GridFlow",
-            PendingDeprecationWarning,
+            "You should use the new standard container property `contents` to modify GridFlow."
+            "API will be removed in version 5.0.",
+            DeprecationWarning,
             stacklevel=2,
         )
         focus_position = self.focus_position
         self.contents = [(new, (WHSettings.GIVEN, self._cell_width)) for new in widgets]
-        if focus_position < len(widgets):  # pylint: disable=consider-using-max-builtin  # pylint bug
+        if focus_position < len(widgets):
             self.focus_position = focus_position
 
-    def _get_cells(self):
-        warnings.warn(
-            "only for backwards compatibility."
-            "You should use the new standard container property `contents` to modify GridFlow",
-            DeprecationWarning,
-            stacklevel=3,
-        )
-        return self.cells
-
-    def _set_cells(self, widgets: Sequence[Widget]):
-        warnings.warn(
-            "only for backwards compatibility."
-            "You should use the new standard container property `contents` to modify GridFlow",
-            DeprecationWarning,
-            stacklevel=3,
-        )
-        self.cells = widgets
-
     @property
     def cell_width(self) -> int:
         """
@@ -202,24 +186,6 @@ class GridFlow(WidgetWrap[Pile], WidgetContainerMixin, WidgetContainerListConten
         self.focus_position = focus_position
         self._cell_width = width
 
-    def _get_cell_width(self) -> int:
-        warnings.warn(
-            f"Method `{self.__class__.__name__}._get_cell_width` is deprecated, "
-            f"please use property `{self.__class__.__name__}.cell_width`",
-            DeprecationWarning,
-            stacklevel=3,
-        )
-        return self.cell_width
-
-    def _set_cell_width(self, width: int) -> None:
-        warnings.warn(
-            f"Method `{self.__class__.__name__}._set_cell_width` is deprecated, "
-            f"please use property `{self.__class__.__name__}.cell_width`",
-            DeprecationWarning,
-            stacklevel=3,
-        )
-        self.cell_width = width
-
     @property
     def contents(self) -> MonitoredFocusList[tuple[Widget, tuple[Literal[WHSettings.GIVEN], int]]]:
         """
@@ -270,8 +236,9 @@ class GridFlow(WidgetWrap[Pile], WidgetContainerMixin, WidgetContainerListConten
         """
         warnings.warn(
             "only for backwards compatibility."
-            "You may also use the new standard container property `focus_position` to set the focus.",
-            PendingDeprecationWarning,
+            "You may also use the new standard container property `focus_position` to set the focus."
+            "API will be removed in version 5.0.",
+            DeprecationWarning,
             stacklevel=2,
         )
         if isinstance(cell, int):
@@ -298,17 +265,6 @@ class GridFlow(WidgetWrap[Pile], WidgetContainerMixin, WidgetContainerListConten
             return None
         return self.contents[self.focus_position][0]
 
-    def _get_focus(self) -> Widget | None:
-        warnings.warn(
-            f"method `{self.__class__.__name__}._get_focus` is deprecated, "
-            f"please use `{self.__class__.__name__}.focus` property",
-            DeprecationWarning,
-            stacklevel=3,
-        )
-        if not self.contents:
-            return None
-        return self.contents[self.focus_position][0]
-
     def get_focus(self):
         """
         Return the widget in focus, for backwards compatibility.
@@ -318,8 +274,9 @@ class GridFlow(WidgetWrap[Pile], WidgetContainerMixin, WidgetContainerListConten
         """
         warnings.warn(
             "only for backwards compatibility."
-            "You may also use the new standard container property `focus` to get the focus.",
-            PendingDeprecationWarning,
+            "You may also use the new standard container property `focus` to get the focus."
+            "API will be removed in version 5.0.",
+            DeprecationWarning,
             stacklevel=2,
         )
         if not self.contents:
@@ -331,8 +288,9 @@ class GridFlow(WidgetWrap[Pile], WidgetContainerMixin, WidgetContainerListConten
         warnings.warn(
             "only for backwards compatibility."
             "You may also use the new standard container property"
-            "`focus` to get the focus and `focus_position` to get/set the cell in focus by index",
-            PendingDeprecationWarning,
+            "`focus` to get the focus and `focus_position` to get/set the cell in focus by index."
+            "API will be removed in version 5.0.",
+            DeprecationWarning,
             stacklevel=2,
         )
         return self.focus
@@ -342,23 +300,10 @@ class GridFlow(WidgetWrap[Pile], WidgetContainerMixin, WidgetContainerListConten
         warnings.warn(
             "only for backwards compatibility."
             "You may also use the new standard container property"
-            "`focus` to get the focus and `focus_position` to get/set the cell in focus by index",
-            PendingDeprecationWarning,
-            stacklevel=2,
-        )
-        for i, (w, _options) in enumerate(self.contents):
-            if cell == w:
-                self.focus_position = i
-                return
-        raise ValueError(f"Widget not found in GridFlow contents: {cell!r}")
-
-    def _set_focus_cell(self, cell: Widget) -> None:
-        warnings.warn(
-            "only for backwards compatibility."
-            "You may also use the new standard container property"
-            "`focus` to get the focus and `focus_position` to get/set the cell in focus by index",
+            "`focus` to get the focus and `focus_position` to get/set the cell in focus by index."
+            "API will be removed in version 5.0.",
             DeprecationWarning,
-            stacklevel=3,
+            stacklevel=2,
         )
         for i, (w, _options) in enumerate(self.contents):
             if cell == w:
@@ -392,38 +337,6 @@ class GridFlow(WidgetWrap[Pile], WidgetContainerMixin, WidgetContainerListConten
             ) from exc
         self.contents.focus = position
 
-    def _get_focus_position(self) -> int | None:
-        warnings.warn(
-            f"method `{self.__class__.__name__}._get_focus_position` is deprecated, "
-            f"please use `{self.__class__.__name__}.focus_position` property",
-            DeprecationWarning,
-            stacklevel=3,
-        )
-        if not self.contents:
-            raise IndexError("No focus_position, GridFlow is empty")
-        return self.contents.focus
-
-    def _set_focus_position(self, position: int) -> None:
-        """
-        Set the widget in focus.
-
-        position -- index of child widget to be made focus
-        """
-        warnings.warn(
-            f"method `{self.__class__.__name__}._set_focus_position` is deprecated, "
-            f"please use `{self.__class__.__name__}.focus_position` property",
-            DeprecationWarning,
-            stacklevel=3,
-        )
-        try:
-            if position < 0 or position >= len(self.contents):
-                raise IndexError(f"No GridFlow child widget at position {position}")
-        except TypeError as exc:
-            raise IndexError(f"No GridFlow child widget at position {position}").with_traceback(
-                exc.__traceback__
-            ) from exc
-        self.contents.focus = position
-
     def _get_maxcol(self, size: tuple[int] | tuple[()]) -> int:
         if size:
             (maxcol,) = size
@@ -540,10 +453,12 @@ class GridFlow(WidgetWrap[Pile], WidgetContainerMixin, WidgetContainerListConten
         Captures focus changes.
         """
         self.get_display_widget(size)
-        key = super().keypress(size, key)
-        if key is None:
-            self._set_focus_from_display_widget()
-        return key
+
+        if (key := super().keypress(size, key)) is not None:
+            return key
+
+        self._set_focus_from_display_widget()
+        return None
 
     def pack(
         self,

urwid/widget/line_box.py

@@ -68,29 +68,29 @@ class LineBox(WidgetDecoration[WrappedWidget], delegate_to_widget_mixin("_wrappe
         │Some text│
         └─────────┘
         >>> print(
-        ...   LineBox(
-        ...     Text("Some text"),
-        ...     tlcorner=LineBox.Symbols.LIGHT.TOP_LEFT_ROUNDED,
-        ...     trcorner=LineBox.Symbols.LIGHT.TOP_RIGHT_ROUNDED,
-        ...     blcorner=LineBox.Symbols.LIGHT.BOTTOM_LEFT_ROUNDED,
-        ...     brcorner=LineBox.Symbols.LIGHT.BOTTOM_RIGHT_ROUNDED,
-        ...   ).render(())
+        ...     LineBox(
+        ...         Text("Some text"),
+        ...         tlcorner=LineBox.Symbols.LIGHT.TOP_LEFT_ROUNDED,
+        ...         trcorner=LineBox.Symbols.LIGHT.TOP_RIGHT_ROUNDED,
+        ...         blcorner=LineBox.Symbols.LIGHT.BOTTOM_LEFT_ROUNDED,
+        ...         brcorner=LineBox.Symbols.LIGHT.BOTTOM_RIGHT_ROUNDED,
+        ...     ).render(())
         ... )
         ╭─────────╮
         │Some text│
         ╰─────────╯
         >>> print(
-        ...   LineBox(
-        ...     Text("Some text"),
-        ...     tline=LineBox.Symbols.HEAVY.HORIZONTAL,
-        ...     bline=LineBox.Symbols.HEAVY.HORIZONTAL,
-        ...     lline=LineBox.Symbols.HEAVY.VERTICAL,
-        ...     rline=LineBox.Symbols.HEAVY.VERTICAL,
-        ...     tlcorner=LineBox.Symbols.HEAVY.TOP_LEFT,
-        ...     trcorner=LineBox.Symbols.HEAVY.TOP_RIGHT,
-        ...     blcorner=LineBox.Symbols.HEAVY.BOTTOM_LEFT,
-        ...     brcorner=LineBox.Symbols.HEAVY.BOTTOM_RIGHT,
-        ...   ).render(())
+        ...     LineBox(
+        ...         Text("Some text"),
+        ...         tline=LineBox.Symbols.HEAVY.HORIZONTAL,
+        ...         bline=LineBox.Symbols.HEAVY.HORIZONTAL,
+        ...         lline=LineBox.Symbols.HEAVY.VERTICAL,
+        ...         rline=LineBox.Symbols.HEAVY.VERTICAL,
+        ...         tlcorner=LineBox.Symbols.HEAVY.TOP_LEFT,
+        ...         trcorner=LineBox.Symbols.HEAVY.TOP_RIGHT,
+        ...         blcorner=LineBox.Symbols.HEAVY.BOTTOM_LEFT,
+        ...         brcorner=LineBox.Symbols.HEAVY.BOTTOM_RIGHT,
+        ...     ).render(())
         ... )
         ┏━━━━━━━━━┓
         ┃Some text┃
@@ -167,6 +167,19 @@ class LineBox(WidgetDecoration[WrappedWidget], delegate_to_widget_mixin("_wrappe
 
         super().__init__(original_widget)
 
+    @property
+    def original_widget(self) -> WrappedWidget:
+        return super().original_widget
+
+    @original_widget.setter
+    def original_widget(self, original_widget: WrappedWidget) -> None:
+        v_index = int(bool(self.tline_widget))  # we care only about top
+        h_index = 1  # constant
+        middle: Columns = typing.cast("Columns", self._wrapped_widget[v_index])
+        _old_widget, options = middle.contents[h_index]
+        middle.contents[h_index] = (original_widget, options)
+        WidgetDecoration.original_widget.fset(self, original_widget)  # pylint: disable=no-member
+
     @property
     def _w(self) -> Pile:
         return self._wrapped_widget

urwid/widget/listbox.py

@@ -26,8 +26,6 @@ import warnings
 from collections.abc import Iterable, Sized
 from contextlib import suppress
 
-from typing_extensions import Protocol, runtime_checkable
-
 from urwid import signals
 from urwid.canvas import CanvasCombine, SolidCanvas
 
@@ -65,8 +63,8 @@ class ListWalkerError(Exception):
     pass
 
 
-@runtime_checkable
-class ScrollSupportingBody(Protocol):
+@typing.runtime_checkable
+class ScrollSupportingBody(typing.Protocol):
     """Protocol for ListWalkers."""
 
     def get_focus(self) -> tuple[Widget, _K]: ...
@@ -78,8 +76,8 @@ class ScrollSupportingBody(Protocol):
     def get_prev(self, position: _K) -> tuple[Widget, _K] | tuple[None, None]: ...
 
 
-@runtime_checkable
-class EstimatedSized(Protocol):
+@typing.runtime_checkable
+class EstimatedSized(typing.Protocol):
     """Widget can estimate it's size.
 
     PEP 424 defines API for memory-efficiency.
@@ -166,15 +164,6 @@ class SimpleListWalker(MonitoredList[_T], ListWalker):
         """
         return self
 
-    def _get_contents(self) -> Self:
-        warnings.warn(
-            f"Method `{self.__class__.__name__}._get_contents` is deprecated, "
-            f"please use property`{self.__class__.__name__}.contents`",
-            DeprecationWarning,
-            stacklevel=3,
-        )
-        return self
-
     def _modified(self) -> None:
         if self.focus >= len(self):
             self.focus = max(0, len(self) - 1)
@@ -426,24 +415,6 @@ class ListBox(Widget, WidgetContainerMixin):
             self.render = nocache_widget_render_instance(self)
         self._invalidate()
 
-    def _get_body(self):
-        warnings.warn(
-            f"Method `{self.__class__.__name__}._get_body` is deprecated, "
-            f"please use property `{self.__class__.__name__}.body`",
-            DeprecationWarning,
-            stacklevel=3,
-        )
-        return self.body
-
-    def _set_body(self, body):
-        warnings.warn(
-            f"Method `{self.__class__.__name__}._set_body` is deprecated, "
-            f"please use property `{self.__class__.__name__}.body`",
-            DeprecationWarning,
-            stacklevel=3,
-        )
-        self.body = body
-
     @property
     def __len__(self) -> Callable[[], int]:
         if isinstance(self._body, Sized):
@@ -873,8 +844,9 @@ class ListBox(Widget, WidgetContainerMixin):
         warnings.warn(
             "only for backwards compatibility."
             "You may also use the new standard container property `focus` to get the focus "
-            "and property `focus_position` to read these values.",
-            PendingDeprecationWarning,
+            "and property `focus_position` to read these values."
+            "API will be removed in version 5.0.",
+            DeprecationWarning,
             stacklevel=2,
         )
         return self._body.get_focus()
@@ -888,15 +860,6 @@ class ListBox(Widget, WidgetContainerMixin):
         """
         return self._body.get_focus()[0]
 
-    def _get_focus(self) -> Widget:
-        warnings.warn(
-            f"method `{self.__class__.__name__}._get_focus` is deprecated, "
-            f"please use `{self.__class__.__name__}.focus` property",
-            DeprecationWarning,
-            stacklevel=3,
-        )
-        return self.focus
-
     def _get_focus_position(self):
         """
         Return the list walker position of the widget in focus. The type
@@ -1054,6 +1017,9 @@ class ListBox(Widget, WidgetContainerMixin):
         self._body.set_focus(focus_pos)
 
         middle, top, bottom = self.calculate_visible((maxcol, maxrow), focus)
+        if middle is None:
+            return None
+
         focus_offset, _focus_widget, focus_pos, focus_rows, _cursor = middle  # pylint: disable=unpacking-non-sequence
         _trim_top, fill_above = top  # pylint: disable=unpacking-non-sequence
         _trim_bottom, fill_below = bottom  # pylint: disable=unpacking-non-sequence
@@ -1231,7 +1197,7 @@ class ListBox(Widget, WidgetContainerMixin):
             # start from closest edge and move inwards
             (pref_col,) = cursor_coords
             if coming_from == "above":
-                attempt_rows = range(0, tgt_rows)
+                attempt_rows = range(tgt_rows)
             else:
                 if coming_from != "below":
                     raise ValueError("must specify coming_from ('above' or 'below') if cursor row is not specified")
@@ -1938,8 +1904,7 @@ class ListBox(Widget, WidgetContainerMixin):
             )
             return False
 
-        handled = w.mouse_event((maxcol,), event, button, col, row - wrow, focus)
-        if handled:
+        if w.mouse_event((maxcol,), event, button, col, row - wrow, focus):
             return True
 
         if is_mouse_press(event):
@@ -1995,8 +1960,8 @@ class ListBox(Widget, WidgetContainerMixin):
         the focus up to the top.  This is the best we can do with
         a minimal list walker implementation.
         """
-        positions_fn = getattr(self._body, "positions", None)
-        if positions_fn:
+
+        if positions_fn := getattr(self._body, "positions", None):
             yield from positions_fn()
             return
 
@@ -2026,8 +1991,8 @@ class ListBox(Widget, WidgetContainerMixin):
         reverse of what `__iter__()` produces, but this is the best we can
         do with a minimal list walker implementation.
         """
-        positions_fn = getattr(self._body, "positions", None)
-        if positions_fn:
+
+        if positions_fn := getattr(self._body, "positions", None):
             yield from positions_fn(reverse=True)
             return