From: John Lindgren <john@jlindgren.net>
Date: Wed, 6 Aug 2025 04:23:40 +0000 (-0400)
Subject: xdg: don't try to reposition unmapped view in timeout handler
X-Git-Url: https://git.mdlowis.com/?a=commitdiff_plain;h=9557091a1a100cb9b2646b5de61f9dbe796053e8;p=proto%2Flabwc.git

xdg: don't try to reposition unmapped view in timeout handler
---

diff --git a/src/xdg.c b/src/xdg.c
index 363bd478..d71a7d93 100644
--- a/src/xdg.c
+++ b/src/xdg.c
@@ -288,6 +288,14 @@ handle_configure_timeout(void *data)
 	view->pending_configure_serial = 0;
 	view->pending_configure_timeout = NULL;
 
+	/*
+	 * No need to do anything else if the view is just being slow to
+	 * map - the map handler will take care of the positioning.
+	 */
+	if (!view->mapped) {
+		return 0; /* ignored per wl_event_loop docs */
+	}
+
 	bool empty_pending = wlr_box_empty(&view->pending);
 	if (empty_pending || view->pending.x != view->current.x
 			|| view->pending.y != view->current.y) {