diff options
author | Drew DeVault <sir@cmpwn.com> | 2017-10-17 09:34:47 -0400 |
---|---|---|
committer | GitHub <noreply@github.com> | 2017-10-17 09:34:47 -0400 |
commit | 9129687ad6dcf162b3af8d91b2fc582f185d0943 (patch) | |
tree | 1d87cbc4577ebfaf81fe00737a8ca3a6a6a155f8 | |
parent | 62371a9974c5c86f151a702042d67cc73dcb4708 (diff) | |
parent | 82076bd772f8d015774d3efbf1e98230757d0980 (diff) |
Merge pull request #280 from acrisci/bug/xdg-shell-pending-different-force
xdg-shell: force configure when pending different
-rw-r--r-- | types/wlr_xdg_shell_v6.c | 23 |
1 files changed, 17 insertions, 6 deletions
diff --git a/types/wlr_xdg_shell_v6.c b/types/wlr_xdg_shell_v6.c index c8c09dc9..f057025a 100644 --- a/types/wlr_xdg_shell_v6.c +++ b/types/wlr_xdg_shell_v6.c @@ -841,6 +841,10 @@ static const struct zxdg_surface_v6_interface zxdg_surface_v6_implementation = { static bool wlr_xdg_surface_v6_toplevel_state_compare( struct wlr_xdg_toplevel_v6 *state) { // is pending state different from current state? + if (!state->base->configured) { + return false; + } + if (state->pending.activated != state->current.activated) { return false; } @@ -1261,42 +1265,49 @@ void wlr_xdg_surface_v6_ping(struct wlr_xdg_surface_v6 *surface) { void wlr_xdg_toplevel_v6_set_size(struct wlr_xdg_surface_v6 *surface, uint32_t width, uint32_t height) { assert(surface->role == WLR_XDG_SURFACE_V6_ROLE_TOPLEVEL); + bool force = + (surface->toplevel_state->pending.width != width || + surface->toplevel_state->pending.height != height); surface->toplevel_state->pending.width = width; surface->toplevel_state->pending.height = height; - wlr_xdg_surface_v6_schedule_configure(surface, false); + wlr_xdg_surface_v6_schedule_configure(surface, force); } void wlr_xdg_toplevel_v6_set_activated(struct wlr_xdg_surface_v6 *surface, bool activated) { assert(surface->role == WLR_XDG_SURFACE_V6_ROLE_TOPLEVEL); + bool force = surface->toplevel_state->pending.activated != activated; surface->toplevel_state->pending.activated = activated; - wlr_xdg_surface_v6_schedule_configure(surface, false); + wlr_xdg_surface_v6_schedule_configure(surface, force); } void wlr_xdg_toplevel_v6_set_maximized(struct wlr_xdg_surface_v6 *surface, bool maximized) { assert(surface->role == WLR_XDG_SURFACE_V6_ROLE_TOPLEVEL); + bool force = surface->toplevel_state->pending.maximized != maximized; surface->toplevel_state->pending.maximized = maximized; - wlr_xdg_surface_v6_schedule_configure(surface, false); + wlr_xdg_surface_v6_schedule_configure(surface, force); } void wlr_xdg_toplevel_v6_set_fullscreen(struct wlr_xdg_surface_v6 *surface, bool fullscreen) { assert(surface->role == WLR_XDG_SURFACE_V6_ROLE_TOPLEVEL); + bool force = surface->toplevel_state->pending.fullscreen != fullscreen; surface->toplevel_state->pending.fullscreen = fullscreen; - wlr_xdg_surface_v6_schedule_configure(surface, false); + wlr_xdg_surface_v6_schedule_configure(surface, force); } void wlr_xdg_toplevel_v6_set_resizing(struct wlr_xdg_surface_v6 *surface, bool resizing) { assert(surface->role == WLR_XDG_SURFACE_V6_ROLE_TOPLEVEL); - surface->toplevel_state->pending.fullscreen = resizing; + bool force = surface->toplevel_state->pending.resizing != resizing; + surface->toplevel_state->pending.resizing = resizing; - wlr_xdg_surface_v6_schedule_configure(surface, false); + wlr_xdg_surface_v6_schedule_configure(surface, force); } void wlr_xdg_toplevel_v6_send_close(struct wlr_xdg_surface_v6 *surface) { |