https://bugzilla.gnome.org/show_bug.cgi?id=777176
Jonas Ådahl <[email protected]> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |[email protected]
--- Comment #6 from Jonas Ådahl <[email protected]> ---
(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._______________________________________________
wayland-bugs mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/wayland-bugs
