Changeset b0ae23f in mainline for uspace/srv/hid/display/window.c

Timestamp:

2023-01-17T15:21:13Z (15 months ago)

Author:

Jiri Svoboda <jiri@…>

Branches:

master, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export

Children:

a0d4afe

Parents:

46a47c0

git-author:

Jiri Svoboda <jiri@…> (2023-01-17 18:21:02)

git-committer:

Jiri Svoboda <jiri@…> (2023-01-17 15:21:13)

Message:

Change the correct pointer's shape when resizing window

The request to resize a window originates from the client. The display
server forces the cursor to a double-arrow shape regardless of whether
it is over the window or not, until the resize is done. This is to
make sure the cursor keeps that shape even if it moves outside of
the current boundaries of the window. With multiple pointers we need
to know which one to change. This is done by passing the pos_id from
button press event that starts the resize all the way to
ds_window_start_resize(). Then it needs to be stored in the window
structure for use when it is time to restore the cursor.

File:

• uspace/srv/hid/display/window.c (modified) (4 diffs)

uspace/srv/hid/display/window.c

@@ -510 +510 @@
  * @param rsztype Resize type (which part of window is being dragged)
  * @param pos Position where mouse button was pressed
+ * @param pos_id Positioning device ID
  */
 static void ds_window_start_resize(ds_window_t *wnd,
-    display_wnd_rsztype_t rsztype, gfx_coord2_t *pos)
+    display_wnd_rsztype_t rsztype, gfx_coord2_t *pos, sysarg_t pos_id)
 {
         ds_seat_t *seat;
@@ -523 +524 @@
                 return;
 
+        /* Determine which seat started the resize */
+        seat = ds_display_seat_by_idev(wnd->display, pos_id);
+        if (seat == NULL)
+                return;
+
         wnd->orig_pos = *pos;
+        wnd->orig_pos_id = pos_id;
         wnd->state = dsw_resizing;
         wnd->rsztype = rsztype;
         wnd->preview_rect = wnd->rect;
 
-        // TODO Multi-seat: need client to tell us which seat started the resize!
-        seat = ds_display_first_seat(wnd->display);
         ctype = display_cursor_from_wrsz(rsztype);
         ds_seat_set_wm_cursor(seat, wnd->display->cursor[ctype]);
@@ -559 +564 @@
         ds_client_post_resize_event(wnd->client, wnd, &nrect);
 
-        // TODO Multi-seat: Need to know which seat started the resize!
-        seat = ds_display_first_seat(wnd->display);
-        ds_seat_set_wm_cursor(seat, NULL);
+        /* Determine which seat started the resize */
+        seat = ds_display_seat_by_idev(wnd->display, wnd->orig_pos_id);
+        if (seat != NULL)
+                ds_seat_set_wm_cursor(seat, NULL);
 
         (void) ds_display_paint(wnd->display, NULL);
@@ -787 +793 @@
  * @param pos Position where the pointer was when the resize started
  *            relative to the window
+ * @param pos_id Positioning device ID
  * @param event Button press event
  */
 void ds_window_resize_req(ds_window_t *wnd, display_wnd_rsztype_t rsztype,
-    gfx_coord2_t *pos)
+    gfx_coord2_t *pos, sysarg_t pos_id)
 {
         gfx_coord2_t orig_pos;
 
-        log_msg(LOG_DEFAULT, LVL_DEBUG, "ds_window_resize_req (%d, %d, %d)",
-            (int) rsztype, (int) pos->x, (int) pos->y);
+        log_msg(LOG_DEFAULT, LVL_DEBUG, "ds_window_resize_req (%d, %d, %d, %d)",
+            (int)rsztype, (int)pos->x, (int)pos->y, (int)pos_id);
 
         gfx_coord2_add(&wnd->dpos, pos, &orig_pos);
-        ds_window_start_resize(wnd, rsztype, &orig_pos);
+        ds_window_start_resize(wnd, rsztype, &orig_pos, pos_id);
 }