Re: [PATCH 1/2] xwayland: Update screen size on output removal

2015-11-17 Thread Olivier Fourdan
Hi - Original Message - > > Or does Xwayland perhaps have a requirement for compositors to have > > no output space to the left and/or right of 0,0? > > I don't know if there's such a written requirement, but reading the code as > you did, I don't think it expect outputs to a have negativ

Re: [PATCH 1/2] xwayland: Update screen size on output removal

2015-11-16 Thread Olivier Fourdan
Hi Pekka, > I see that the above is old code just moved and I don't really know Yes, it is old code that I simply moved up in the source so I could reuse it without adding a forward declaration :-) > how Xwayland works here, but that code looks strange to me. I don't > know what useful meaning

Re: [PATCH 1/2] xwayland: Update screen size on output removal

2015-11-15 Thread Pekka Paalanen
On Fri, 13 Nov 2015 17:57:22 +0100 Olivier Fourdan wrote: > When unplugging an output, it's still listed in xrandr and the size > of the root window still includes the removed output. > > The XRandR output should be destroyed when its Wayland counterpart is > destroyed and the screen dimensions

[PATCH 1/2] xwayland: Update screen size on output removal

2015-11-13 Thread Olivier Fourdan
When unplugging an output, it's still listed in xrandr and the size of the root window still includes the removed output. The XRandR output should be destroyed when its Wayland counterpart is destroyed and the screen dimensions must be updated in both the done and the destroy handler. Bugzilla: h