Merge pull request #6598 from Textualize/screen-padding

fix screen padding crash

Author: willmcgugan

CHANGELOG.md

@@ -10,6 +10,7 @@ and this project adheres to [Semantic Versioning](http://semver.org/).
 ### Fixed
 
 - Fixed parsing Kitty extended keys with multiple codepoints https://github.com/Textualize/textual/pull/6592
+- Fixed crash when clicking in the Screen's padding https://github.com/Textualize/textual/pull/6598
 
 ### Changed
 

src/textual/_compositor.py

@@ -923,11 +923,8 @@ def get_widget_and_offset_at(
         x -= region.x + gutter_left
         y -= region.y + gutter_right
 
-        if y < 0:
-            return None, None
-
-        if x < 0:
-            return widget, Offset(0, y)
+        if x < 0 or y < 0:
+            return widget, Offset(max(0, x), max(0, y))
 
         visible_screen_stack.set(widget.app._background_screens)
         line = widget.render_line(y)

src/textual/pilot.py

@@ -438,7 +438,7 @@ async def _post_mouse_events(
         )
 
         offset = Offset(message_arguments["x"], message_arguments["y"])
-        if offset not in screen.region:
+        if offset not in screen.size.region:
             raise OutOfBounds(
                 "Target offset is outside of currently-visible screen region."
             )

src/textual/screen.py

@@ -1838,8 +1838,11 @@ def _forward_event(self, event: events.Event) -> None:
                     if select_offset is not None:
                         content_widget = select_widget
                         content_offset = select_offset
-                        assert isinstance(content_widget.parent, Widget)
-                        container = content_widget.parent
+                        container = (
+                            content_widget
+                            if isinstance(content_widget, Screen)
+                            else content_widget.parent
+                        )
                     else:
                         content_widget = None
                         container = select_widget
@@ -1884,6 +1887,7 @@ def _forward_event(self, event: events.Event) -> None:
                 select_widget, select_offset = self.get_widget_and_offset_at(
                     event.x, event.y
                 )
+
                 if (
                     select_widget is not None
                     and select_widget.allow_select
@@ -1893,8 +1897,11 @@ def _forward_event(self, event: events.Event) -> None:
                     if select_offset is not None:
                         content_widget = select_widget
                         content_offset = select_offset
-                        assert isinstance(content_widget.parent, Widget)
-                        container = content_widget.parent
+                        container = (
+                            content_widget
+                            if isinstance(content_widget, Screen)
+                            else content_widget.parent
+                        )
                     else:
                         content_widget = None
                         container = select_widget