mirrors/labwc
1
0
Fork 0
mirror of https://github.com/labwc/labwc.git synced 2026-03-06 01:40:15 -05:00
Code Issues Projects Releases Packages Wiki Activity Actions

xwayland: call foreign-toplevel-destroy on unmap

Browse source 
Some xwayland clients leave unmapped child views around, typically when a
dialog window is closed. Although handle_destroy() is not called for
these, we have to call foreign-toplevel-destroy to avoid panels and the
like incorrecly showing them.

Backport of 62ae87bbd9
This commit is contained in:
Johan Malm 2022-05-17 18:10:29 +01:00 • committed by Johan Malm
parent 83f8e4952d
commit 12718e5e63
1 changed files with 21 additions and 6 deletions
Download patch file Download diff file Expand all files Collapse all files

27
src/xwayland.c
View file

@ -78,6 +78,18 @@ handle_unmap(struct wl_listener *listener, void *data)
{ {
struct view *view = wl_container_of(listener, view, unmap); struct view *view = wl_container_of(listener, view, unmap);
view->impl->unmap(view); view->impl->unmap(view);
/*
* Some xwayland clients leave unmapped child views around, typically
* when a dialog window is closed. Although handle_destroy() is not
* called for these, we have to deal with them as such in terms of the
* foreign-toplevel protocol to avoid panels and the like still showing
* them.
*/
if (view->toplevel_handle) {
wlr_foreign_toplevel_handle_v1_destroy(view->toplevel_handle);
view->toplevel_handle = NULL;
}
} }
static void static void
@ -308,9 +320,11 @@ map(struct view *view)
ssd_create(view); ssd_create(view);
} }
if (!view->been_mapped) { if (!view->toplevel_handle) {
foreign_toplevel_handle_create(view); foreign_toplevel_handle_create(view);
}
if (!view->been_mapped) {
if (!view->maximized && !view->fullscreen) { if (!view->maximized && !view->fullscreen) {
struct wlr_box box = struct wlr_box box =
output_usable_area_from_cursor_coords(view->server); output_usable_area_from_cursor_coords(view->server);
@ -338,12 +352,13 @@ map(struct view *view)
static void static void
unmap(struct view *view) unmap(struct view *view)
{ {
if (view->mapped) { if (!view->mapped) {
view->mapped = false; return;
damage_all_outputs(view->server);
wl_list_remove(&view->commit.link);
desktop_focus_topmost_mapped_view(view->server);
} }
view->mapped = false;
damage_all_outputs(view->server);
wl_list_remove(&view->commit.link);
desktop_focus_topmost_mapped_view(view->server);
} }
static void static void