<div dir="ltr">On Tue, Jul 29, 2014 at 9:16 PM, Bill Spitzak <span dir="ltr"><<a href="mailto:spitzak@gmail.com" target="_blank">spitzak@gmail.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div class="">On 07/29/2014 11:40 AM, Manuel Bachmann wrote:<br>
<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
When creating a xdg_surface, the surface will not be mapped (i.e. shown)<br>
by desktop-shell anymore. It will only be if xdg_surface_present() has<br>
been called once.<br>
</blockquote>
<br></div>
There seems to be a design goal in Wayland to prevent clients from making surfaces that they never map. So it would be better if creation + commit of a surface did the same thing as present. Also this does not break existing clients.<br>
</blockquote><br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
There is nothing special about the first time the surface wants attention (other than historical legacy). The desktop should be allowed to turn this into a notification just like it would on subsequent calls.<br>
</blockquote><div><br></div><div>Agreed. Especially if you start an application, but it's slow to start, and you have typed into the current window or have navigated away from it since, you should get a popup instead of the window immediately mapped. This is known as "focus stealing prevention".<br>
</div><div><br></div><div>Unfortunately, the protocol as Manuel mocked up doesn't have the event timestamp. This is required so we can track when the surface was intended to be presented.<br><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<div class=""><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
If called twice, or more, the request will send an event to<br>
desktop-shell, so it can display a notification.<br>
</blockquote>
<br></div>
This is not controlled by a count, but by whether a window is already visible or already in the notification state. Clients should be able to send a lot of these in a row. They cannot reliably test if they are invisible and send the request only then, as there is a race condition.<br>
</blockquote><div><br></div><div>Yes, this is what's intended.<br><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
I also think the term "present" is not a great idea. This should be exactly the same as "raise" or "show" or "activate" or any number of other terms, but I have never seen the word "present" used before. I would reuse an existing term. One reason is to prevent somebody else from adding a redundant api for that term, because they did not realize "present" is the thing they are looking for.<br>
</blockquote><div><br>For reference, the name was taken from gtk_window_present. I am fine with alternate names for the request, including "activate". I am not comfortable with "raise" or "show", though, as the request is designed to *present* the surface to the window, whenever possible. This may imply a workspace switch, too. The name "raise" or "show" simply says to me about the stacking order or the mapped state, but this may just be my X11 experience talking.<br>
 </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
______________________________<u></u>_________________<br>
wayland-devel mailing list<br>
<a href="mailto:wayland-devel@lists.freedesktop.org" target="_blank">wayland-devel@lists.<u></u>freedesktop.org</a><br>
<a href="http://lists.freedesktop.org/mailman/listinfo/wayland-devel" target="_blank">http://lists.freedesktop.org/<u></u>mailman/listinfo/wayland-devel</a><br>
</blockquote></div><br><br clear="all"><br>-- <br>  Jasper<br>
</div></div>