[PATCH wayland-protocols] xdg-output: add a transform example for the logical size

[Jonas Ådahl]

On Mon, Jul 16, 2018 at 02:45:11PM -0400, Simon Ser wrote:
> Hi Jonas,
> 
> What do you think of this patch?

Thanks. Merged after shortening the line:

   65cc109..3f28298  master -> master


Jonas

> 
> Thanks,
> 
> Simon
> 
> On July 9, 2018 3:25 PM, Olivier Fourdan <fourdan at gmail.com> wrote:
> > Hi,
> >
> > On Mon, 9 Jul 2018 at 11:09, Simon Ser <contact at emersion.fr> wrote:
> > >
> > > Bump: this is the source of bugs in xwayland [1].
> > >
> > > [1]: https://lists.x.org/archives/xorg-devel/2018-July/057285.html
> > >
> > > On May 18, 2018 9:40 PM, Simon Ser <contact at emersion.fr> wrote:
> > > > Signed-off-by: Simon Ser <contact at emersion.fr>
> > > > ---
> > > >
> > > > I believe it's easy to understand that transformations are not applied to
> > > > the logical size.
> > > >
> > > >  unstable/xdg-output/xdg-output-unstable-v1.xml | 3 +++
> > > >  1 file changed, 3 insertions(+)
> > > >
> > > > diff --git a/unstable/xdg-output/xdg-output-unstable-v1.xml b/unstable/xdg-output/xdg-output-unstable-v1.xml
> > > > index bd9a8a8..320c0a0 100644
> > > > --- a/unstable/xdg-output/xdg-output-unstable-v1.xml
> > > > +++ b/unstable/xdg-output/xdg-output-unstable-v1.xml
> > > > @@ -135,6 +135,9 @@
> > > >       - A compositor using a fractional scale of 1.5 will advertise a
> > > >         logical size to 2560×1620.
> > > >
> > > > +     For example, for a wl_output mode 1920×1080 and a 90 degree rotation, the
> > > > +     compositor will advertise a logical size of 1080x1920.
> > > > +
> > > >       The logical_size event is sent after creating an xdg_output
> > > >       (see xdg_output_manager.get_xdg_output) and whenever the logical
> > > >       size of the output changes, either as a result of a change in the
> > > > --
> > > > 2.17.0
> >
> > Reviewed-by: Olivier Fourdan <ofourdan at redhat.com>
> >
> > That r-b doesn't apply to the Xwayland patch though, to which I 'll
> > reply in the relevant thread.
> >
> > Cheers,
> > Olivier
> 
> _______________________________________________
> wayland-devel mailing list
> wayland-devel at lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/wayland-devel