<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Feb 3, 2016 at 1:05 PM, Jasper St. Pierre <span dir="ltr"><<a href="mailto:jstpierre@mecheye.net" target="_blank">jstpierre@mecheye.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><span class="">On Wed, Feb 3, 2016 at 1:00 PM, Bill Spitzak <<a href="mailto:spitzak@gmail.com">spitzak@gmail.com</a>> wrote:<br>
><br>
><br>
> On Wed, Feb 3, 2016 at 11:36 AM, Jasper St. Pierre <<a href="mailto:jstpierre@mecheye.net">jstpierre@mecheye.net</a>><br>
> wrote:<br>
>><br>
>> set_parent was moved to xdg_toplevel. Perhaps that's a good idea,<br>
>> perhaps it's not. This does mean that a tooltip's parent can never<br>
>> change, and a popup's parent can never change. That can help for<br>
>> getting grab semantics right, though it might be an idea if we say<br>
>> that set_parent on a popup / tooltip simply emit error if a parent is<br>
>> already set. I'm -1 to that, though.<br>
><br>
><br>
> xdg_toplevel *must* support the ability to change the parent.<br>
<br>
</span>Somehow I get the feeling you're not even reading the words I'm<br>
writing. xdg_toplevel still has a set_parent method, I said it was<br>
moved *to* xdg_toplevel.<br></blockquote><div><br></div><div>Sorry I am confused then as to what the text "This does mean that a tooltip's parent can never change, and a popup's parent can never change". I think I misread it as "if we use xdg_surface set_parent, you cannot change the parent", thus implying that xdg_surface set_parent cannot change it. I think what you actually meant was "the current scheme does not allow you to change the parent, this proposal *does* allow the parent to change".</div><div><br></div><div>If there are problems with making the grabs work, I think it is ok if attempting to change the parent is either an error or ignored for surfaces with the popup role.</div><div><br></div><div><br></div></div><br></div></div>