https://bugzilla.gnome.org/show_bug.cgi?id=783901
Olivier Fourdan <[email protected]> changed:
What |Removed |Added
----------------------------------------------------------------------------
Attachment #366644|reviewed |none
status| |
Attachment #366644|0 |1
is obsolete| |
--- Comment #40 from Olivier Fourdan <[email protected]> ---
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._______________________________________________
wayland-bugs mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/wayland-bugs
