[PATCH wayland-protocols] xdg-output: Clarify that output positions refer to the top-left corner

[Olivier Fourdan]

Hi,

On Fri, May 10, 2019 at 6:53 PM William Wold <william.wold at canonical.com> wrote:
>
> I already sent this two weeks ago, but I didn't get the subject right
> and I didn't get any response. Hopefully this will work better. I

Yeap, sorry I missed it :/

> believe this is just clarifying what the protocol already meant (but I
> can't be sure, on account of it not currently being clear).

Yes, that's what was assumed, although wl_output::geometry event
doesn't specify this either:

https://wayland.freedesktop.org/docs/html/apa.html#protocol-spec-wl_output

> ---
>  unstable/xdg-output/xdg-output-unstable-v1.xml | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/unstable/xdg-output/xdg-output-unstable-v1.xml
> b/unstable/xdg-output/xdg-output-unstable-v1.xml
> index ccbfe1c..3176520 100644
> --- a/unstable/xdg-output/xdg-output-unstable-v1.xml
> +++ b/unstable/xdg-output/xdg-output-unstable-v1.xml
> @@ -94,17 +94,17 @@
>
>      <event name="logical_position">
>        <description summary="position of the output within the global
> compositor space">
> -       The position event describes the location of the wl_output within
> -       the global compositor space.
> +       The position event describes the location of the top-left corner
> +       of the wl_output within the global compositor space.
>
>         The logical_position event is sent after creating an xdg_output
>         (see xdg_output_manager.get_xdg_output) and whenever the location
>         of the output changes within the global compositor space.
>        </description>
>        <arg name="x" type="int"
> -          summary="x position within the global compositor space"/>
> +          summary="x position of the output's left edge within the
> global compositor space"/>
>        <arg name="y" type="int"
> -          summary="y position within the global compositor space"/>
> +          summary="y position of the output's top edge within the
> global compositor space"/>
>      </event>
>
>      <event name="logical_size">
> --
> 2.20.1

LGTM.

Reviewed-by: Olivier Fourdan <ofourdan at redhat.com>

Cheers

Olivier