<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] gedit killed by protocol error "Invalid anchor rectangle size""
href="https://bugzilla.gnome.org/show_bug.cgi?id=777176">bug 777176</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;">CC</td>
<td>
</td>
<td>jadahl@gmail.com
</td>
</tr></table>
<p>
<div>
<b><a class="bz_bug_link
bz_status_NEW "
title="NEW - [wayland] gedit killed by protocol error "Invalid anchor rectangle size""
href="https://bugzilla.gnome.org/show_bug.cgi?id=777176#c6">Comment # 6</a>
on <a class="bz_bug_link
bz_status_NEW "
title="NEW - [wayland] gedit killed by protocol error "Invalid anchor rectangle size""
href="https://bugzilla.gnome.org/show_bug.cgi?id=777176">bug 777176</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>(In reply to Olivier Fourdan from <a href="show_bug.cgi?id=777176#c5">comment #5</a>)
<span class="quote">> Yes, as you say, xdg-shell v6 positionner definition makes it clear that the
> rectangle should not be 0:
>
>
> <a href="https://cgit.freedesktop.org/wayland/wayland-protocols/tree/unstable/xdg">https://cgit.freedesktop.org/wayland/wayland-protocols/tree/unstable/xdg</a>-
> shell/xdg-shell-unstable-v6.xml#n115
>
> For an xdg_positioner object to be considered complete, it must have a
> non-zero size set by set_size, and a non-zero anchor rectangle set by
> set_anchor_rect. Passing an incomplete xdg_positioner object when
> positioning a surface raises an error.
>
> So the same error occurs with weston as well.
>
> There is a possibility that we can do, though, if the given size is zero, we
> use 1 as with <span class=""><a href="attachment.cgi?id=343383&action=diff" name="attach_343383" title="[PATCH] wayland: avoid 0 width/height anchor rectangle">attachment 343383</a> <a href="attachment.cgi?id=343383&action=edit" title="[PATCH] wayland: avoid 0 width/height anchor rectangle">[details]</a></span> <a href='review?bug=777176&attachment=343383'>[review]</a> [review] but then compensate by
> moving the position by 1 pixel in the opposite direction.
>
> Definitely a hack, but that would give the expected result without breaking
> the existing rules.</span >
The problem with that is that it'd be offset by 2 pixels if it flipped. Then
again, if this is supposed to open at the "|" cursor (maybe it should still be
a non-empty anchor that it should flip around, i.e. the width of the "|" cursor
thing. So, I think the current patch is the best we can do for now, but we
should still probably allow empty-size anchor in the next xdg-positioner
version.</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>