wayland: Always call Wayland_SetWindowBordered when showing the window

Otherwise our windows have no window decoration on compositors that
support xdg-decoration-unstable-v1, but default to client-side mode.

Contrary to what the comment was stating, there is nothing in the protocol
that would make redundant calls to zxdg_toplevel_decoration_v1::set_mode
problematic.
This commit is contained in:
Sebastian Krzyszkowiak 2021-08-02 05:49:35 +02:00 committed by Sam Lantinga
parent cfcdfb7be9
commit 9bb24ad0cf
1 changed files with 2 additions and 5 deletions

View File

@ -756,12 +756,9 @@ void Wayland_ShowWindow(_THIS, SDL_Window *window)
/* Unlike the rest of window state we have to set this _after_ flushing the /* Unlike the rest of window state we have to set this _after_ flushing the
* display, because we need to create the decorations before possibly hiding * display, because we need to create the decorations before possibly hiding
* them immediately afterward. But don't call it redundantly, the protocol * them immediately afterward.
* may not interpret a redundant call nicely and cause weird stuff to happen
*/ */
if (window->flags & SDL_WINDOW_BORDERLESS) { Wayland_SetWindowBordered(_this, window, !(window->flags & SDL_WINDOW_BORDERLESS));
Wayland_SetWindowBordered(_this, window, SDL_FALSE);
}
/* We're finally done putting the window together, raise if possible */ /* We're finally done putting the window together, raise if possible */
if (c->activation_manager) { if (c->activation_manager) {