[Wayland-bugs] [Bug 777176] [wayland] gedit killed by protocol error "Invalid anchor rectangle size"

[gtk+ (GNOME Bugzilla)]

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

Jonas Ådahl <jadahl at gmail.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |jadahl at gmail.com

--- Comment #6 from Jonas Ådahl <jadahl at gmail.com> ---
(In reply to Olivier Fourdan from comment #5)
> Yes, as you say, xdg-shell v6 positionner definition makes it clear that the
> rectangle should not be 0:
> 
>  
> https://cgit.freedesktop.org/wayland/wayland-protocols/tree/unstable/xdg-
> 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 attachment 343383 [details] [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.

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.

-- 
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/20170113/5acaf9f0/attachment.html>