urwid 2.6.9__tar.gz → 2.6.11__tar.gz

urwid-2.6.11/.editorconfig

@@ -0,0 +1,22 @@
+root = true
+
+[*]
+charset = utf-8
+end_of_line = lf
+indent_style = space
+insert_final_newline = true
+trim_trailing_whitespace = true
+
+[*.{py,pyi,pyx,pxd}]
+indent_size = 4
+max_line_length = 120
+
+[*.{yml,yaml,json,ini,toml}]
+indent_size = 2
+max_line_length = 120
+
+[*.{rst,md}]
+indent_size = 2
+
+[{Makefile,Makefile.in}]
+indent_style = tab

{urwid-2.6.9 → urwid-2.6.11}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: urwid
-Version: 2.6.9
+Version: 2.6.11
 Summary: A full-featured console (xterm et al.) user interface library
 Home-page: https://urwid.org/
 Author-email: Ian Ward <ian@excess.org>

urwid-2.6.11/black-requirements.txt

@@ -0,0 +1 @@
+black==24.4.0

urwid-2.6.11/docs/examples/bigtext.py.xdotool

@@ -0,0 +1,3 @@
+windowsize --usehints $RXVTWINDOWID 39 21
+key --window $RXVTWINDOWID Down Down Down Down Down Down Down Down Return
+key --window $RXVTWINDOWID Up Up Up Return

{urwid-2.6.9 → urwid-2.6.11}/docs/examples/browse.py

@@ -6,5 +6,5 @@ import os
 
 import real_browse
 
-os.chdir("/usr/share/doc/python")
+os.chdir("/usr/share/doc/python3")
 real_browse.main()

{urwid-2.6.9 → urwid-2.6.11}/docs/examples/graph.py

@@ -209,8 +209,30 @@ class GraphView(urwid.WidgetWrap):
         bg = urwid.AttrMap(urwid.SolidFill("▒"), "screen edge")
         shadow = urwid.AttrMap(urwid.SolidFill(" "), "main shadow")
 
-        bg = urwid.Overlay(shadow, bg, ("fixed left", 3), ("fixed right", 1), ("fixed top", 2), ("fixed bottom", 1))
-        w = urwid.Overlay(w, bg, ("fixed left", 2), ("fixed right", 3), ("fixed top", 1), ("fixed bottom", 2))
+        bg = urwid.Overlay(
+            shadow,
+            bg,
+            align=urwid.LEFT,
+            width=urwid.RELATIVE_100,
+            valign=urwid.TOP,
+            height=urwid.RELATIVE_100,
+            left=3,
+            right=1,
+            top=2,
+            bottom=1,
+        )
+        w = urwid.Overlay(
+            w,
+            bg,
+            align=urwid.LEFT,
+            width=urwid.RELATIVE_100,
+            valign=urwid.TOP,
+            height=urwid.RELATIVE_100,
+            left=2,
+            right=3,
+            top=1,
+            bottom=2,
+        )
         return w
 
     def bar_graph(self, smooth=False):
@@ -288,10 +310,10 @@ class GraphView(urwid.WidgetWrap):
     def main_window(self):
         self.graph = self.bar_graph()
         self.graph_wrap = urwid.WidgetWrap(self.graph)
-        vline = urwid.AttrMap(urwid.SolidFill("\u2502"), "line")
+        vline = urwid.AttrMap(urwid.SolidFill("│"), "line")
         c = self.graph_controls()
         w = urwid.Columns([(urwid.WEIGHT, 2, self.graph_wrap), (1, vline), c], dividechars=1, focus_column=2)
-        w = urwid.Padding(w, ("fixed left", 1), ("fixed right", 0))
+        w = urwid.Padding(w, urwid.LEFT, left=1)
         w = urwid.AttrMap(w, "body")
         w = urwid.LineBox(w)
         w = urwid.AttrMap(w, "line")

{urwid-2.6.9 → urwid-2.6.11}/docs/examples/real_browse.py

@@ -274,7 +274,14 @@ class DirectoryBrowser:
         self.listbox.offset_rows = 1
         self.footer = urwid.AttrMap(urwid.Text(self.footer_text), "foot")
         self.view = urwid.Frame(
-            urwid.AttrMap(urwid.ScrollBar(self.listbox), "body"),
+            urwid.AttrMap(
+                urwid.ScrollBar(
+                    self.listbox,
+                    thumb_char=urwid.ScrollBar.Symbols.FULL_BLOCK,
+                    trough_char=urwid.ScrollBar.Symbols.LITE_SHADE,
+                ),
+                "body",
+            ),
             header=urwid.AttrMap(self.header, "head"),
             footer=self.footer,
         )
@@ -379,9 +386,9 @@ def escape_filename_sh_ansic(name: str) -> str:
 SPLIT_RE = re.compile(r"[a-zA-Z]+|\d+")
 
 
-def alphabetize(s):
+def alphabetize(s: str) -> list[str]:
     L = []
-    for isdigit, group in itertools.groupby(SPLIT_RE.findall(s), key=lambda x: x.isdigit()):
+    for isdigit, group in itertools.groupby(SPLIT_RE.findall(s), key=str.isdigit):
         if isdigit:
             L.extend(("", int(n)) for n in group)
         else:

urwid-2.6.11/docs/examples/tour.py.xdotool

@@ -0,0 +1,2 @@
+windowsize --usehints $RXVTWINDOWID 39 25
+key --window $RXVTWINDOWID Return End

{urwid-2.6.9 → urwid-2.6.11}/examples/browse.py

@@ -274,7 +274,14 @@ class DirectoryBrowser:
         self.listbox.offset_rows = 1
         self.footer = urwid.AttrMap(urwid.Text(self.footer_text), "foot")
         self.view = urwid.Frame(
-            urwid.AttrMap(urwid.ScrollBar(self.listbox), "body"),
+            urwid.AttrMap(
+                urwid.ScrollBar(
+                    self.listbox,
+                    thumb_char=urwid.ScrollBar.Symbols.FULL_BLOCK,
+                    trough_char=urwid.ScrollBar.Symbols.LITE_SHADE,
+                ),
+                "body",
+            ),
             header=urwid.AttrMap(self.header, "head"),
             footer=self.footer,
         )
@@ -379,9 +386,9 @@ def escape_filename_sh_ansic(name: str) -> str:
 SPLIT_RE = re.compile(r"[a-zA-Z]+|\d+")
 
 
-def alphabetize(s):
+def alphabetize(s: str) -> list[str]:
     L = []
-    for isdigit, group in itertools.groupby(SPLIT_RE.findall(s), key=lambda x: x.isdigit()):
+    for isdigit, group in itertools.groupby(SPLIT_RE.findall(s), key=str.isdigit):
         if isdigit:
             L.extend(("", int(n)) for n in group)
         else:

{urwid-2.6.9 → urwid-2.6.11}/examples/dialog.py

@@ -68,8 +68,8 @@ class DialogDisplay:
         w = self.frame
 
         # pad area around listbox
-        w = urwid.Padding(w, ("fixed left", 2), ("fixed right", 2))
-        w = urwid.Filler(w, ("fixed top", 1), ("fixed bottom", 1))
+        w = urwid.Padding(w, urwid.LEFT, left=2, right=2)
+        w = urwid.Filler(w, urwid.TOP, urwid.RELATIVE_100, top=1, bottom=1)
         w = urwid.AttrMap(w, "body")
 
         # "shadow" effect

{urwid-2.6.9 → urwid-2.6.11}/examples/graph.py

@@ -209,8 +209,30 @@ class GraphView(urwid.WidgetWrap):
         bg = urwid.AttrMap(urwid.SolidFill("▒"), "screen edge")
         shadow = urwid.AttrMap(urwid.SolidFill(" "), "main shadow")
 
-        bg = urwid.Overlay(shadow, bg, ("fixed left", 3), ("fixed right", 1), ("fixed top", 2), ("fixed bottom", 1))
-        w = urwid.Overlay(w, bg, ("fixed left", 2), ("fixed right", 3), ("fixed top", 1), ("fixed bottom", 2))
+        bg = urwid.Overlay(
+            shadow,
+            bg,
+            align=urwid.LEFT,
+            width=urwid.RELATIVE_100,
+            valign=urwid.TOP,
+            height=urwid.RELATIVE_100,
+            left=3,
+            right=1,
+            top=2,
+            bottom=1,
+        )
+        w = urwid.Overlay(
+            w,
+            bg,
+            align=urwid.LEFT,
+            width=urwid.RELATIVE_100,
+            valign=urwid.TOP,
+            height=urwid.RELATIVE_100,
+            left=2,
+            right=3,
+            top=1,
+            bottom=2,
+        )
         return w
 
     def bar_graph(self, smooth=False):
@@ -288,10 +310,10 @@ class GraphView(urwid.WidgetWrap):
     def main_window(self):
         self.graph = self.bar_graph()
         self.graph_wrap = urwid.WidgetWrap(self.graph)
-        vline = urwid.AttrMap(urwid.SolidFill("\u2502"), "line")
+        vline = urwid.AttrMap(urwid.SolidFill("│"), "line")
         c = self.graph_controls()
         w = urwid.Columns([(urwid.WEIGHT, 2, self.graph_wrap), (1, vline), c], dividechars=1, focus_column=2)
-        w = urwid.Padding(w, ("fixed left", 1), ("fixed right", 0))
+        w = urwid.Padding(w, urwid.LEFT, left=1)
         w = urwid.AttrMap(w, "body")
         w = urwid.LineBox(w)
         w = urwid.AttrMap(w, "line")

{urwid-2.6.9 → urwid-2.6.11}/pyproject.toml

@@ -2,9 +2,9 @@
 # Minimum requirements for the build system to execute.
 # PEP 508 specifications for PEP 518.
 requires = [
-    "setuptools >= 61.0.0",
-    "setuptools_scm[toml]>=7.0",
-    "wheel",
+  "setuptools >= 61.0.0",
+  "setuptools_scm[toml]>=7.0",
+  "wheel",
 ]
 build-backend="setuptools.build_meta"
 
@@ -179,7 +179,7 @@ extend-select = [
 ]
 extend-ignore = [
   "PLR6301", "E203",
-    # refactor rules (too many statements/arguments/branches)
+  # refactor rules (too many statements/arguments/branches)
   "PLR0904", "PLR0911", "PLR0912", "PLR0913", "PLR0914", "PLR0915", "PLR0917", "PLR2004",
   "PLC0415",  # We have a lot of imports outside of top-level
   "RET504",  # Unnecessary variable assignment before return statement

urwid-2.6.11/ruff-requirements.txt

@@ -0,0 +1 @@
+ruff==0.4.1

{urwid-2.6.9 → urwid-2.6.11}/tests/test_scrollable.py

@@ -1,10 +1,14 @@
 from __future__ import annotations
 
 import string
+import typing
 import unittest
 
 import urwid
 
+if typing.TYPE_CHECKING:
+    from collections.abc import Iterable
+
 LGPL_HEADER = """
 Copyright (C) <year>  <name of author>
 
@@ -264,15 +268,13 @@ class TestScrollBarListBox(unittest.TestCase):
 
         widget.keypress(reduced_size, "end")
 
-        # Here we have ListBox issue: "end" really scroll not to the "dead end"
-        # in case of the bottom focus position is multiline
         self.assertEqual(
             (
-                "GNU Lesser General Public               ",
                 "License along with this library; if     ",
                 "not, write to the Free Software         ",
                 "Foundation, Inc., 51 Franklin Street,   ",
-                "Fifth Floor, Boston, MA  02110-1301    █",
+                "Fifth Floor, Boston, MA  02110-1301     ",
+                "USA                                    █",
             ),
             widget.render(reduced_size).decoded_text,
         )
@@ -344,3 +346,54 @@ class TestScrollBarListBox(unittest.TestCase):
         widget.keypress(reduced_size, "down")
 
         self.assertEqual(pos_1_down_rendered, widget.render(reduced_size).decoded_text)
+
+    def test_hinted_len(self):
+        class HintedWalker(urwid.ListWalker):
+            def __init__(self, items: Iterable[str]) -> None:
+                self.items: tuple[str] = tuple(items)
+                self.focus = 0
+                self.requested_numbers: set[int] = set()
+
+            def __length_hint__(self) -> int:
+                return len(self.items)
+
+            def __getitem__(self, item: int) -> urwid.Text:
+                self.requested_numbers.add(item)
+                return urwid.Text(self.items[item])
+
+            def set_focus(self, item: int) -> None:
+                self.focus = item
+
+            def next_position(self, position: int) -> int:
+                if position + 1 < len(self.items):
+                    return position + 1
+                raise IndexError
+
+            def prev_position(self, position: int) -> int:
+                if position - 1 >= 0:
+                    return position - 1
+                raise IndexError
+
+        widget = urwid.ScrollBar(urwid.ListBox(HintedWalker((f"Line {idx:02}") for idx in range(1, 51))))
+        size = (10, 10)
+        widget.original_widget.focus_position = 19
+        self.assertEqual(
+            (
+                "Line 16   ",
+                "Line 17   ",
+                "Line 18   ",
+                "Line 19  █",
+                "Line 20  █",
+                "Line 21   ",
+                "Line 22   ",
+                "Line 23   ",
+                "Line 24   ",
+                "Line 25   ",
+            ),
+            widget.render(size).decoded_text,
+        )
+        self.assertNotIn(
+            30,
+            widget.original_widget.body.requested_numbers,
+            "Requested index out of range [0, last shown]. This means not relative scroll bar built.",
+        )

{urwid-2.6.9 → urwid-2.6.11}/tox.ini

@@ -7,8 +7,8 @@ usedevelop = true
 deps = -r {toxinidir}/test_requirements.txt
 
 commands =
-    coverage run -m unittest discover -s tests -v
-    coverage report
+  coverage run -m unittest discover -s tests -v
+  coverage report
 
 [testenv:readme]
 deps =
@@ -21,7 +21,7 @@ commands =
 [testenv:isort]
 skip_install = true
 deps =
-    isort
+  isort
 commands =
   isort .
 
@@ -41,18 +41,18 @@ commands = ruff check .
 
 [testenv:docs]
 deps =
-    -r {toxinidir}/test_requirements.txt
-    PyGObject
-    sphinx
-    sphinx-github-changelog
+  -r {toxinidir}/test_requirements.txt
+  PyGObject
+  sphinx
+  sphinx-github-changelog
 commands = sphinx-build docs/ build/documentation
 
 [testenv:pylint]
 depends = black,isort
 deps =
-    -r {toxinidir}/test_requirements.txt
-    PyGObject
-    pylint
+  -r {toxinidir}/test_requirements.txt
+  PyGObject
+  pylint
 commands = pylint urwid
 
 [testenv:refurb]

{urwid-2.6.9 → urwid-2.6.11}/urwid/version.py

@@ -12,5 +12,5 @@ __version__: str
 __version_tuple__: VERSION_TUPLE
 version_tuple: VERSION_TUPLE
 
-__version__ = version = '2.6.9'
-__version_tuple__ = version_tuple = (2, 6, 9)
+__version__ = version = '2.6.11'
+__version_tuple__ = version_tuple = (2, 6, 11)

{urwid-2.6.9 → urwid-2.6.11}/urwid/widget/listbox.py

@@ -20,6 +20,7 @@
 
 from __future__ import annotations
 
+import operator
 import typing
 import warnings
 from collections.abc import Iterable, Sized
@@ -77,6 +78,18 @@ class ScrollSupportingBody(Protocol):
     def get_prev(self, position: _K) -> tuple[Widget, _K] | tuple[None, None]: ...
 
 
+@runtime_checkable
+class EstimatedSized(Protocol):
+    """Widget can estimate it's size.
+
+    PEP 424 defines API for memory-efficiency.
+    For the ListBox it's a sign of the limited body length.
+    The main use-case is lazy-load, where real length calculation is expensive.
+    """
+
+    def __length_hint__(self) -> int: ...
+
+
 class ListWalker(metaclass=signals.MetaSignals):  # pylint: disable=no-member, unsubscriptable-object
     # mixin not named as mixin
     signals: typing.ClassVar[list[str]] = ["modified"]
@@ -431,11 +444,18 @@ class ListBox(Widget, WidgetContainerMixin):
         )
         self.body = body
 
-    def __len__(self) -> int:
+    @property
+    def __len__(self) -> Callable[[], int]:
         if isinstance(self._body, Sized):
-            return len(self._body)
+            return self._body.__len__
         raise AttributeError(f"{self._body.__class__.__name__} is not Sized")
 
+    @property
+    def __length_hint__(self) -> Callable[[], int]:  # pylint: disable=invalid-length-hint-returned
+        if isinstance(self._body, (Sized, EstimatedSized)):
+            return lambda: operator.length_hint(self._body)
+        raise AttributeError(f'{self._body.__class__.__name__} is not Sized and do not implement "__length_hint__"')
+
     def calculate_visible(
         self,
         size: tuple[int, int],
@@ -577,9 +597,9 @@ class ListBox(Widget, WidgetContainerMixin):
         if not isinstance(self._body, ScrollSupportingBody):
             raise ListBoxError(f"{self} body do not implement methods required for scrolling protocol")
 
-        if not isinstance(self._body, (Sized, TreeWalker)):
+        if not isinstance(self._body, (Sized, EstimatedSized, TreeWalker)):
             raise ListBoxError(
-                f"{self} body is not a Sized and not a TreeWalker."
+                f"{self} body is not a Sized, can not estimate it's size and not a TreeWalker."
                 f"Scroll is not allowed due to risk of infinite cycle of widgets load."
             )
 
@@ -643,7 +663,7 @@ class ListBox(Widget, WidgetContainerMixin):
 
     def require_relative_scroll(self, size: tuple[int, int], focus: bool = False) -> bool:
         """Widget require relative scroll due to performance limitations of real lines count calculation."""
-        return isinstance(self._body, Sized) and (size[1] * 3 < len(self))
+        return isinstance(self._body, (Sized, EstimatedSized)) and (size[1] * 3 < operator.length_hint(self.body))
 
     def get_first_visible_pos(self, size: tuple[int, int], focus: bool = False) -> int:
         self._check_support_scrolling()

{urwid-2.6.9 → urwid-2.6.11}/urwid/widget/monitored_list.py

@@ -24,9 +24,9 @@ import typing
 import warnings
 
 if typing.TYPE_CHECKING:
-    from collections.abc import Callable, Collection, Iterator
+    from collections.abc import Callable, Collection, Iterable, Iterator
 
-    from typing_extensions import Concatenate, ParamSpec
+    from typing_extensions import Concatenate, ParamSpec, Self
 
     ArgSpec = ParamSpec("ArgSpec")
     Ret = typing.TypeVar("Ret")
@@ -88,23 +88,69 @@ class MonitoredList(typing.List[_T], typing.Generic[_T]):
         for item in self:
             yield None, item
 
-    __add__ = _call_modified(list.__add__)  # type: ignore[assignment]  # magic like old __super__
-    __delitem__ = _call_modified(list.__delitem__)  # type: ignore[assignment]  # magic like old __super__
-
-    __iadd__ = _call_modified(list.__iadd__)  # type: ignore[assignment]  # magic like old __super__
-    __imul__ = _call_modified(list.__imul__)  # type: ignore[assignment]  # magic like old __super__
-    __rmul__ = _call_modified(list.__rmul__)  # type: ignore[assignment]  # magic like old __super__
-    __setitem__ = _call_modified(list.__setitem__)  # type: ignore[assignment]  # magic like old __super__
-
-    append = _call_modified(list.append)  # type: ignore[assignment]  # magic like old __super__
-    extend = _call_modified(list.extend)  # type: ignore[assignment]  # magic like old __super__
-    insert = _call_modified(list.insert)  # type: ignore[assignment]  # magic like old __super__
-    pop = _call_modified(list.pop)  # type: ignore[assignment]  # magic like old __super__
-    remove = _call_modified(list.remove)  # type: ignore[assignment]  # magic like old __super__
-    reverse = _call_modified(list.reverse)  # type: ignore[assignment]  # magic like old __super__
-    sort = _call_modified(list.sort)  # type: ignore[assignment]  # magic like old __super__
-    if hasattr(list, "clear"):
-        clear = _call_modified(list.clear)  # type: ignore[assignment]  # magic like old __super__
+    @_call_modified
+    def __add__(self, __value: list[_T]) -> Self:
+        return super().__add__(__value)
+
+    @_call_modified
+    def __delitem__(self, __key: typing.SupportsIndex | slice) -> None:
+        super().__delitem__(__key)
+
+    @_call_modified
+    def __iadd__(self, __value: Iterable[_T]) -> Self:
+        return super().__iadd__(__value)
+
+    @_call_modified
+    def __rmul__(self, __value: typing.SupportsIndex) -> Self:
+        return super().__rmul__(__value)
+
+    @_call_modified
+    def __imul__(self, __value: typing.SupportsIndex) -> Self:
+        return super().__imul__(__value)
+
+    @typing.overload
+    @_call_modified
+    def __setitem__(self, __key: typing.SupportsIndex, __value: _T) -> None: ...
+
+    @typing.overload
+    @_call_modified
+    def __setitem__(self, __key: slice, __value: Iterable[_T]) -> None: ...
+
+    @_call_modified
+    def __setitem__(self, __key: typing.SupportsIndex | slice, __value: _T | Iterable[_T]) -> None:
+        super().__setitem__(__key, __value)
+
+    @_call_modified
+    def append(self, __object: _T) -> None:
+        super().append(__object)
+
+    @_call_modified
+    def extend(self, __iterable: Iterable[_T]) -> None:
+        super().extend(__iterable)
+
+    @_call_modified
+    def pop(self, __index: typing.SupportsIndex = -1) -> _T:
+        return super().pop(__index)
+
+    @_call_modified
+    def insert(self, __index: typing.SupportsIndex, __object: _T) -> None:
+        super().insert(__index, __object)
+
+    @_call_modified
+    def remove(self, __value: _T) -> None:
+        super().remove(__value)
+
+    @_call_modified
+    def reverse(self) -> None:
+        super().reverse()
+
+    @_call_modified
+    def sort(self, *, key: Callable[[_T], typing.Any] | None = None, reverse: bool = False) -> None:
+        super().sort(key=key, reverse=reverse)
+
+    @_call_modified
+    def clear(self) -> None:
+        super().clear()
 
 
 class MonitoredFocusList(MonitoredList[_T], typing.Generic[_T]):

{urwid-2.6.9 → urwid-2.6.11}/urwid/widget/padding.py

@@ -40,12 +40,14 @@ class Padding(WidgetDecoration[WrappedWidget], typing.Generic[WrappedWidget]):
         self,
         w: WrappedWidget,
         align: (
-            Literal["left", "center", "right"] | Align | tuple[Literal["relative", WHSettings.RELATIVE], int]
+            Literal["left", "center", "right"]
+            | Align
+            | tuple[Literal["relative", WHSettings.RELATIVE, "fixed left", "fixed right"], int]
         ) = Align.LEFT,
         width: (
             int
             | Literal["pack", "clip", WHSettings.PACK, WHSettings.CLIP]
-            | tuple[Literal["relative", WHSettings.RELATIVE], int]
+            | tuple[Literal["relative", WHSettings.RELATIVE, "fixed left", "fixed right"], int]
         ) = RELATIVE_100,
         min_width: int | None = None,
         left: int = 0,

{urwid-2.6.9 → urwid-2.6.11}/urwid/widget/popup.py

@@ -24,7 +24,7 @@ import typing
 
 from urwid.canvas import CompositeCanvas
 
-from .constants import Sizing
+from .constants import Align, Sizing, VAlign
 from .overlay import Overlay
 from .widget import delegate_to_widget_mixin
 from .widget_decoration import WidgetDecoration
@@ -86,8 +86,7 @@ class PopUpLauncher(delegate_to_widget_mixin("_original_widget"), WidgetDecorati
 
 
 class PopUpTarget(WidgetDecoration[WrappedWidget]):
-    # FIXME: this whole class is a terrible hack and must be fixed
-    # when layout and rendering are separated
+    # FIXME: this whole class is a terrible hack and must be fixed when layout and rendering are separated
     _sizing = frozenset((Sizing.BOX,))
     _selectable = True
 
@@ -105,19 +104,23 @@ class PopUpTarget(WidgetDecoration[WrappedWidget]):
             if self._pop_up != w:
                 self._pop_up = w
                 self._current_widget = Overlay(
-                    w,
-                    self._original_widget,
-                    ("fixed left", left),
-                    overlay_width,
-                    ("fixed top", top),
-                    overlay_height,
+                    top_w=w,
+                    bottom_w=self._original_widget,
+                    align=Align.LEFT,
+                    width=overlay_width,
+                    valign=VAlign.TOP,
+                    height=overlay_height,
+                    left=left,
+                    top=top,
                 )
             else:
                 self._current_widget.set_overlay_parameters(
-                    ("fixed left", left),
-                    overlay_width,
-                    ("fixed top", top),
-                    overlay_height,
+                    align=Align.LEFT,
+                    width=overlay_width,
+                    valign=VAlign.TOP,
+                    height=overlay_height,
+                    left=left,
+                    top=top,
                 )
         else:
             self._pop_up = None