From: "Anna (navi) Figueiredo Gomes" navi@vlhl.dev
in tiling compositors, or those with decorations always on, having popups (like menu items or on-hover tooltips) be set as toplevels causes them to become unusable. Instead, if a HWND is owned by another window that has a wayland surface, create the surface as a xdg_popup of the owner.
Signed-off-by: Anna (navi) Figueiredo Gomes navi@vlhl.dev --- dlls/winewayland.drv/wayland_surface.c | 92 +++++++++++++++++++++++++- dlls/winewayland.drv/waylanddrv.h | 4 ++ dlls/winewayland.drv/window.c | 55 ++++++++++++--- 3 files changed, 139 insertions(+), 12 deletions(-)
diff --git a/dlls/winewayland.drv/wayland_surface.c b/dlls/winewayland.drv/wayland_surface.c index 39c3976cdfd..96c30a34423 100644 --- a/dlls/winewayland.drv/wayland_surface.c +++ b/dlls/winewayland.drv/wayland_surface.c @@ -131,6 +131,43 @@ static const struct xdg_toplevel_listener xdg_toplevel_listener = xdg_toplevel_handle_close };
+static void xdg_popup_handle_configure(void *data, + struct xdg_popup *xdg_popup, + int32_t x, int32_t y, + int32_t width, int32_t height) +{ + struct wayland_surface *surface; + HWND hwnd = data; + + TRACE("hwnd=%p %dx%d+%d+%d\n", hwnd, width, height, x, y); + + if (!(surface = wayland_surface_lock_hwnd(hwnd))) return; + + if (surface->xdg_popup == xdg_popup) + { + surface->pending.width = width; + surface->pending.height = height; + } + + pthread_mutex_unlock(&surface->mutex); +} + +static void xdg_popup_handle_done(void *data, struct xdg_popup *xdg_popup) +{ + NtUserPostMessage((HWND)data, WM_SYSCOMMAND, SC_CLOSE, 0); +} + +/* Unhandled event: xdg_popup_reposition is never called */ +static void xdg_popup_handle_repositioned(void *data, struct xdg_popup *xdg_popup, uint32_t token) +{} + +static const struct xdg_popup_listener xdg_popup_listener = +{ + xdg_popup_handle_configure, + xdg_popup_handle_done, + xdg_popup_handle_repositioned, +}; + /********************************************************************** * wayland_surface_create * @@ -212,6 +249,12 @@ void wayland_surface_destroy(struct wayland_surface *surface) surface->xdg_toplevel = NULL; }
+ if (surface->xdg_popup) + { + xdg_popup_destroy(surface->xdg_popup); + surface->xdg_popup = NULL; + } + if (surface->xdg_surface) { xdg_surface_destroy(surface->xdg_surface); @@ -267,6 +310,47 @@ err: ERR("Failed to assign toplevel role to wayland surface\n"); }
+/********************************************************************** + * wayland_surface_make_toplevel + * + * Gives the popup role to a plain wayland surface. + */ +void wayland_surface_make_popup(struct wayland_surface *surface, + struct wayland_surface *parent, + const RECT *visible) +{ + struct xdg_positioner *positioner; + int32_t width = visible->right - visible->left; + int32_t height = visible->bottom - visible->top; + if (width <= 0 || height <= 0) goto err; + + TRACE("surface=%p\n", surface); + + surface->xdg_surface = + xdg_wm_base_get_xdg_surface(process_wayland.xdg_wm_base, surface->wl_surface); + if (!surface->xdg_surface) goto err; + xdg_surface_add_listener(surface->xdg_surface, &xdg_surface_listener, surface->hwnd); + + + positioner = xdg_wm_base_create_positioner(process_wayland.xdg_wm_base); + xdg_positioner_set_size(positioner, width, height); + xdg_positioner_set_anchor_rect(positioner, visible->left, visible->top, width, height); + + surface->xdg_popup = xdg_surface_get_popup(surface->xdg_surface, parent->xdg_surface, positioner); + xdg_positioner_destroy(positioner); + if (!surface->xdg_popup) goto err; + xdg_popup_add_listener(surface->xdg_popup, &xdg_popup_listener, surface->hwnd); + + wl_surface_commit(surface->wl_surface); + wl_display_flush(process_wayland.wl_display); + + return; + +err: + wayland_surface_clear_role(surface); + ERR("Failed to assign popup role to wayland surface\n"); +} + /********************************************************************** * wayland_surface_clear_role * @@ -284,6 +368,12 @@ void wayland_surface_clear_role(struct wayland_surface *surface) surface->xdg_toplevel = NULL; }
+ if (surface->xdg_popup) + { + xdg_popup_destroy(surface->xdg_popup); + surface->xdg_popup = NULL; + } + if (surface->xdg_surface) { xdg_surface_destroy(surface->xdg_surface); @@ -538,7 +628,7 @@ BOOL wayland_surface_reconfigure(struct wayland_surface *surface) struct wayland_window_config *window = &surface->window; int win_width, win_height, width, height;
- if (!surface->xdg_toplevel) return TRUE; + if (!surface->xdg_toplevel && !surface->xdg_popup) return TRUE;
win_width = surface->window.rect.right - surface->window.rect.left; win_height = surface->window.rect.bottom - surface->window.rect.top; diff --git a/dlls/winewayland.drv/waylanddrv.h b/dlls/winewayland.drv/waylanddrv.h index ae719a3d483..0a031c33143 100644 --- a/dlls/winewayland.drv/waylanddrv.h +++ b/dlls/winewayland.drv/waylanddrv.h @@ -195,6 +195,7 @@ struct wayland_surface struct wl_surface *wl_surface; struct xdg_surface *xdg_surface; struct xdg_toplevel *xdg_toplevel; + struct xdg_popup *xdg_popup; struct wp_viewport *wp_viewport; pthread_mutex_t mutex; struct wayland_surface_config pending, requested, processing, current; @@ -239,6 +240,9 @@ void wayland_output_use_xdg_extension(struct wayland_output *output); struct wayland_surface *wayland_surface_create(HWND hwnd); void wayland_surface_destroy(struct wayland_surface *surface); void wayland_surface_make_toplevel(struct wayland_surface *surface); +void wayland_surface_make_popup(struct wayland_surface *surface, + struct wayland_surface *parent, + const RECT *visible); void wayland_surface_clear_role(struct wayland_surface *surface); void wayland_surface_attach_shm(struct wayland_surface *surface, struct wayland_shm_buffer *shm_buffer, diff --git a/dlls/winewayland.drv/window.c b/dlls/winewayland.drv/window.c index 94bd580358b..a4e3ff93e27 100644 --- a/dlls/winewayland.drv/window.c +++ b/dlls/winewayland.drv/window.c @@ -209,10 +209,31 @@ static void reapply_cursor_clipping(void) NtUserSetThreadDpiAwarenessContext(context); }
+static struct wayland_win_data *get_parent_data(HWND handle) +{ + HWND parent = NtUserGetWindowRelative(handle, GW_OWNER); + struct wayland_win_data *data; + + if (parent == 0) return NULL; + + TRACE("parent hwnd: %p\n", parent); + + data = wayland_win_data_get(parent); + if (!data) return NULL; + + if (!data->wayland_surface || !data->wayland_surface->xdg_surface) { + wayland_win_data_release(data); + return NULL; + } + + return data; +} + static void wayland_win_data_update_wayland_surface(struct wayland_win_data *data, const RECT *visible_rect) { struct wayland_surface *surface = data->wayland_surface; HWND parent = NtUserGetAncestor(data->hwnd, GA_PARENT); + struct wayland_win_data *parent_data; BOOL visible, xdg_visible; WCHAR text[1024];
@@ -230,9 +251,8 @@ static void wayland_win_data_update_wayland_surface(struct wayland_win_data *dat
/* Otherwise ensure that we have a wayland surface. */ if (!surface && !(surface = wayland_surface_create(data->hwnd))) return; - visible = (NtUserGetWindowLongW(data->hwnd, GWL_STYLE) & WS_VISIBLE) == WS_VISIBLE; - xdg_visible = surface->xdg_toplevel != NULL; + xdg_visible = surface->xdg_toplevel != NULL || surface->xdg_popup != NULL;
pthread_mutex_lock(&surface->mutex);
@@ -240,17 +260,30 @@ static void wayland_win_data_update_wayland_surface(struct wayland_win_data *dat { /* If we have a pre-existing surface ensure it has no role. */ if (data->wayland_surface) wayland_surface_clear_role(surface); - /* If the window is a visible toplevel make it a wayland - * xdg_toplevel. Otherwise keep it role-less to avoid polluting the - * compositor with empty xdg_toplevels. */ + /* If the window is visible give it a wayland xdg role. + * Otherwise keep it role-less to avoid polluting the + * compositor with empty xdg_toplevels or xdg_popups. */ if (visible) { - wayland_surface_make_toplevel(surface); - if (surface->xdg_toplevel) + /* If the window has a parent that has a wayland surface, + * give it the popup role, so we can position it properly, + * specially in tiling compositors or those with decorations + * enable. Otherwise, make it a toplevel. + * */ + if ((parent_data = get_parent_data(data->hwnd))) { - if (!NtUserInternalGetWindowText(data->hwnd, text, ARRAY_SIZE(text))) - text[0] = 0; - wayland_surface_set_title(surface, text); + wayland_surface_make_popup(surface, parent_data->wayland_surface, visible_rect); + wayland_win_data_release(parent_data); + } + else + { + wayland_surface_make_toplevel(surface); + if (surface->xdg_toplevel) + { + if (!NtUserInternalGetWindowText(data->hwnd, text, ARRAY_SIZE(text))) + text[0] = 0; + wayland_surface_set_title(surface, text); + } } } } @@ -508,7 +541,7 @@ static void wayland_configure_window(HWND hwnd)
if (!(surface = wayland_surface_lock_hwnd(hwnd))) return;
- if (!surface->xdg_toplevel) + if (!surface->xdg_toplevel && !surface->xdg_popup) { TRACE("missing xdg_toplevel, returning\n"); pthread_mutex_unlock(&surface->mutex);