Changes in / [a3086a4:9a664b6d] in mainline

File:

• uspace/srv/hid/display/seat.c (modified) (2 diffs)

uspace/srv/hid/display/seat.c

@@ -1 +1 @@
 /*
- * Copyright (c) 2024 Jiri Svoboda
+ * Copyright (c) 2023 Jiri Svoboda
  * All rights reserved.
  *
@@ -510 +510 @@
 errno_t ds_seat_post_pos_event(ds_seat_t *seat, pos_event_t *event)
 {
-        ds_window_t *pwindow;
-        ds_window_t *cwindow;
+        ds_window_t *wnd;
         errno_t rc;
 
-        /* Window under pointer */
-        pwindow = ds_display_window_by_pos(seat->display, &seat->pntpos);
-
-        /* Current window: popup or focused */
-        cwindow = seat->popup;
-        if (cwindow == NULL)
-                cwindow = seat->focus;
-
-        /*
-         * Deliver move and release event to current window if different
-         * from pwindow
-         */
-        if (event->type != POS_PRESS && cwindow != NULL &&
-            cwindow != pwindow) {
-                rc = ds_window_post_pos_event(cwindow, event);
-                if (rc != EOK)
-                        return rc;
-        }
-
-        if (pwindow != NULL) {
+        wnd = ds_display_window_by_pos(seat->display, &seat->pntpos);
+
+        /* Deliver event to popup window. */
+        if (seat->popup != NULL && event->type != POS_PRESS) {
+                rc = ds_window_post_pos_event(seat->popup, event);
+                if (rc != EOK)
+                        return rc;
+        }
+
+        if (seat->focus != wnd && seat->focus != NULL) {
+                rc = ds_window_post_pos_event(seat->focus, event);
+                if (rc != EOK)
+                        return rc;
+
+                /* Only deliver release events to the focused window */
+                if (event->type == POS_RELEASE)
+                        return EOK;
+        }
+
+        if (wnd != NULL) {
                 /* Moving over a window */
-                ds_seat_set_client_cursor(seat, pwindow->cursor);
-
-                rc = ds_window_post_pos_event(pwindow, event);
-                if (rc != EOK)
-                        return rc;
+                ds_seat_set_client_cursor(seat, wnd->cursor);
+
+                /*
+                 * Only deliver event if we didn't already deliver it
+                 * to the same window above.
+                 */
+                if (wnd != seat->popup || event->type == POS_PRESS) {
+                        rc = ds_window_post_pos_event(wnd, event);
+                        if (rc != EOK)
+                                return rc;
+                }
         } else {
                 /* Not over a window */
-                ds_seat_set_client_cursor(seat,
-                    seat->display->cursor[dcurs_arrow]);
+                ds_seat_set_client_cursor(seat, seat->display->cursor[dcurs_arrow]);
         }
 
         /* Click outside popup window */
-        if (event->type == POS_PRESS && pwindow != seat->popup) {
+        if (event->type == POS_PRESS && wnd != seat->popup) {
                 /* Close popup window */
                 ds_seat_set_popup(seat, NULL);