<html>
<head>
<base href="https://bugzilla.gnome.org/" />
</head>
<body><span class="vcard"><a href="page.cgi?id=describeuser.html&login=jadahl%40gmail.com" title="Jonas Ådahl <jadahl@gmail.com>"> <span class="fn">Jonas Ådahl</span></a>
</span> changed
<a class="bz_bug_link
bz_status_NEW "
title="NEW - wayland: new window momentarily displays on wrong monitor"
href="https://bugzilla.gnome.org/show_bug.cgi?id=782183">bug 782183</a>
<br>
<table border="1" cellspacing="0" cellpadding="8">
<tr>
<th>What</th>
<th>Removed</th>
<th>Added</th>
</tr>
<tr>
<td style="text-align:right;">Attachment #351955 status</td>
<td>reviewed
</td>
<td>accepted-commit_now
</td>
</tr></table>
<p>
<div>
<b><a class="bz_bug_link
bz_status_NEW "
title="NEW - wayland: new window momentarily displays on wrong monitor"
href="https://bugzilla.gnome.org/show_bug.cgi?id=782183#c17">Comment # 17</a>
on <a class="bz_bug_link
bz_status_NEW "
title="NEW - wayland: new window momentarily displays on wrong monitor"
href="https://bugzilla.gnome.org/show_bug.cgi?id=782183">bug 782183</a>
from <span class="vcard"><a href="page.cgi?id=describeuser.html&login=jadahl%40gmail.com" title="Jonas Ådahl <jadahl@gmail.com>"> <span class="fn">Jonas Ådahl</span></a>
</span></b>
<pre>Review of <span class=""><a href="attachment.cgi?id=351955&action=diff" name="attach_351955" title="[PATCH] wayland: place window if maximized before placement">attachment 351955</a> <a href="attachment.cgi?id=351955&action=edit" title="[PATCH] wayland: place window if maximized before placement">[details]</a></span> <a href='review?bug=782183&attachment=351955'>[review]</a>:
::: src/wayland/meta-wayland-xdg-shell.c
@@ +351,3 @@
+ /* Make sure the window is placed first, and if it's already placed,
+ * this has no effect...
+ */
We have been discussing this issue on IRC, and concluded that its probably not
the right time to fix things "too" properly, and as this work around fixes the
issue AFAIK without introducing any new issues, I think it's best to land it.
This comment seems a bit unnecessary though, as it states what is stated below.
The reason why is probably a better thing to document here, but its also
documented in the commit message so, maybe its not necessary anyway.
BTW, I think the commit message is a bit incorrect. Nothing is causing anything
to be "showing early", as showing still will strictly be delayed until a buffer
is attached.</pre>
</div>
</p>
<hr>
<span>You are receiving this mail because:</span>
<ul>
<li>You are on the CC list for the bug.</li>
</ul>
</body>
</html>