<div dir="ltr"><div>Hi,</div><div><br></div><div>I think un-reparenting would involve reparenting to the whatever the parent was before reparenting in the first place. I haven't given this too much thought and I don't know if this case ever happens in practice but figured I'd comment about it since Pekka mentioned possibly reviewing it soon.</div><div><br></div><div>Thanks,</div><div>Scott<br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Mar 29, 2018 at 5:17 AM, Pekka Paalanen <span dir="ltr"><<a href="mailto:ppaalanen@gmail.com" target="_blank">ppaalanen@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Sun, 18 Mar 2018 12:22:15 -0600<br>
Scott Moreau <<a href="mailto:oreaus@gmail.com">oreaus@gmail.com</a>> wrote:<br>
<br>
> Xwayland windows might be created with a different override redirect<br>
> flag than is given on map or configure notify. This causes confusion<br>
> about whether a window should be treated as override redirect or not.<br>
> Here we handle the changing override redirect flag in relevant notify<br>
> handlers so windows are treated appropriately. In particular, this<br>
> fixes positioning menus in clients like VLC, though it is not a<br>
> complete fix. If the window is moved, the menus are still positioned<br>
> as if the window wasn't moved.<br>
> ---<br>
>  xwayland/window-manager.c | 43 ++++++++++++++++++++++++++++++<wbr>+++++--------<br>
>  1 file changed, 35 insertions(+), 8 deletions(-)<br>
<br>
</span>Hi,<br>
<br>
this patch looks big enough that I'd consider it more a new feature<br>
than just a bug fix, so I'd like to postpone this after the 4.0.0<br>
release.<br>
<br>
I've marked this as deferred in Patchwork.<br>
<br>
<br>
Thanks,<br>
pq<br>
</blockquote></div><br></div>