<span></span>On Wednesday, 30 September 2015, Derek Foreman <<a href="mailto:derekf@osg.samsung.com">derekf@osg.samsung.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 30/09/15 12:01 PM, Bill Spitzak wrote:<br>
> My main concern is that programmers are going to search for the word<br>
> "raise" to figure out how to raise windows. They are not going to search<br>
> for "present" or "indicate_activity" or any other bogus word. I suspect<br>
> that if you don't name this "raise" then a function called "raise" will<br>
> be added as an extension very soon, and probably break all your careful<br>
> plans of allowing the compositor to do something other than raise.<br>
<br>
Well, X calls similar functionality "UrgencyHint"...  That's actually<br>
more descriptive than imperative.  The idea is that we let the<br>
compositor know the surface needs attention, not that we tell it what to do.<br>
<br>
Programmers shouldn't be trying to figure out how to raise a window<br>
anyway, they should be searching for words like notify.  But if we put<br>
the word "raise" prominently in the documentation, I guess google should<br>
sort it out for them.  ;)<br>
<br>
In any event, I dislike "present" less than I dislike "raise", so I'm<br>
happy enough with the patch as presented.  You do raise a good point though.<br>
<br>
I'm done.  Really.  Others can continue to fight about this if they so<br>
choose, I've had my say.</blockquote><div><br></div><div> <font size="2"><span style="background-color:rgba(255,255,255,0)">+1 from me, with optional bike shed to request_attention or note_urgency, or anything of their ilk. I don't like 'present' overlapping with the meaning of showing buffers on screen. Urgency / requesting attention have prior art in X11/ICCCM/EWMH. Anyone designing protocols is unlikely to be unaware of this, will have it pointed out to them in review, and can grep the docs for 'raise' anyway.</span></font></div><div><font size="2"><span style="background-color:rgba(255,255,255,0)"><br></span></font></div><div><font size="2"><span style="background-color:rgba(255,255,255,0)">Pending resolution of zero-as-invalid-serial, subsequent flattening of requests, documentation that the extension relies on this behaviour:</span></font></div><div><font size="2"><span style="background-color:rgba(255,255,255,0)">Reviewed-by: Daniel Stone <<a href="mailto:daniels@collabora.com">daniels@collabora.com</a><span></span>></span></font></div><div><font size="2"><span style="background-color:rgba(255,255,255,0)"><br></span></font></div><div><font size="2"><span style="background-color:rgba(255,255,255,0)">(On the assumption that any serial passed will be from xdg-shell or similar; input serials would require a wl_seat to be passed, but if you're reacting to input, then you by definition have the user's attention.)</span></font></div><div><font size="2"><span style="background-color:rgba(255,255,255,0)"><br></span></font></div><div><font size="2"><span style="background-color:rgba(255,255,255,0)">Cheers,</span></font></div><div><font size="2"><span style="background-color:rgba(255,255,255,0)">Daniel</span></font></div>