[Wayland-bugs] [Bug 783901] Restoring maximized window state in Wayland results in tiny window

[mutter (GNOME Bugzilla)]

https://bugzilla.gnome.org/show_bug.cgi?id=783901

Olivier Fourdan <ofourdan at redhat.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #366644|reviewed                    |none
             status|                            |
 Attachment #366644|0                           |1
        is obsolete|                            |

--- Comment #40 from Olivier Fourdan <ofourdan at redhat.com> ---
Created attachment 366645
  --> https://bugzilla.gnome.org/attachment.cgi?id=366645&action=edit
[PATCH 3/3 v5] wayland: update location prior to maximize

(In reply to Jonas Ådahl from comment #39)
> Review of attachment 366644 [details] [review]:
> 
> ::: src/wayland/meta-window-wayland.c
> @@ +237,3 @@
> +      window->rect.x = configured_x;
> +      window->rect.y = configured_y;
> +      can_move_now = TRUE;
> 
> Not exactly what I meant. can_move_now being true will make the x/y = ..
> above happen later in this function (under "if (can_move_now)". Thus should
> be enough to just do can_move_now = TRUE; here.

Ah blighmey, right!

Note: I kept the curly braces on purpose for readability, even though these are
not strictly necessary as there is only one statement left.

-- 
You are receiving this mail because:
You are on the CC list for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/wayland-bugs/attachments/20180111/7ad53368/attachment-0001.html>