[prev in list] [next in list] [prev in thread] [next in thread]
List: kde-panel-devel
Subject: Re: Raising windows on Wayland
From: David Edmundson <david () davidedmundson ! co ! uk>
Date: 2018-07-05 10:19:11
Message-ID: CAGeFrHDrCiHrDY98w5aUYiHE=wh=AWMRDZ10pyCj2cuRUxM9_A () mail ! gmail ! com
[Download RAW message or body]
On Tue, Jul 3, 2018 at 7:45 PM, Martin Fl=C3=B6ser <mgraesslin@kde.org> wro=
te:
> Am 2018-07-03 18:58, schrieb David Edmundson:
>
>>
>>>
>> Ideally this needs to be standardized so that it does not only work
>>> with our applications but with any.
>>>
>>
>> Even within just KDE apps, we have code like QDesktopServices spawning
>> links in code we can't manipulate. Anything non standard, simply isn't
>> worth merging.
>>
>> But I don't think standardising anything would be too hard.
>>
>> Effectively we want something like xdg-foreign which exists. The
>> difference is that it lasts temporarily and it's not manipulating
>> parents.
>>
>
> xdg-foreign is pretty much the "token" idea :-)
>
> Cool, I think we're on the same page. I'll take it up.
>
>> There are two interesting questions:
>>
>> Do we want/need to support wayland window sending activating an
>> xwayland window?
>>
>
> short term: yes, long term: no. Ideally would be a mapping to the startup
> notifications used on X11.
>
>
>> Would we need a serial of the user event (xdg_popup style)? It would
>> which kinda matches the X event timestamps?
>> It's trickier as you'd need a protocol to generate a UUID for both
>> that and the surface to sidechannel, but it's not impossible.
>>
>
> I don't think that the serial is needed as the compositor would only pass
> on focus if the app which generated the token has currently the focus.
> Given that the serial does not provide any additional advantage. And I
> don't like just using the serial as that could be guessed.
>
> Just to clarify a point. I didn't mean just using the serial as a token.
If it was needed I meant clients would need to pass seat + serial as args
along with the surface to our get_token_from_server request.
but I also didn't think it was useful.
> if the app which generated the token has currently the focus
We might need to tweak that. kickoff will close and lose focus when you
click on an app before that app is launched.
This will probably be something up to the compositor, rather than something
we need to worry about in the spec.
-----------
X11 startup system is also used for notifying the UI, via taskbar or
whatever else about the application starting up before it maps a surface.
Is that something we want to bring in? (IMHO no)
David
[Attachment #3 (text/html)]
<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Jul \
3, 2018 at 7:45 PM, Martin Flöser <span dir="ltr"><<a \
href="mailto:mgraesslin@kde.org" target="_blank">mgraesslin@kde.org</a>></span> \
wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px \
#ccc solid;padding-left:1ex"><span>Am 2018-07-03 18:58, schrieb David Edmundson:<br> \
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc \
solid;padding-left:1ex"><blockquote class="gmail_quote" style="margin:0 0 0 \
.8ex;border-left:1px #ccc solid;padding-left:1ex"> <br>
</blockquote>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc \
solid;padding-left:1ex"> Ideally this needs to be standardized so that it does not \
only work<br> with our applications but with any.<br>
</blockquote>
<br>
Even within just KDE apps, we have code like QDesktopServices spawning<br>
links in code we can't manipulate. Anything non standard, simply isn't<br>
worth merging.<br>
<br>
But I don't think standardising anything would be too hard.<br>
<br>
Effectively we want something like xdg-foreign which exists. The<br>
difference is that it lasts temporarily and it's not manipulating<br>
parents.<br>
</blockquote>
<br></span>
xdg-foreign is pretty much the "token" idea :-)<span><br>
<br></span></blockquote><div>Cool, I think we're on the same page. I'll take \
it up.<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 \
.8ex;border-left:1px #ccc solid;padding-left:1ex"><span> <blockquote \
class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc \
solid;padding-left:1ex"> <br>
There are two interesting questions:<br>
<br>
Do we want/need to support wayland window sending activating an<br>
xwayland window?<br>
</blockquote>
<br></span>
short term: yes, long term: no. Ideally would be a mapping to the startup \
notifications used on X11.<span><br> <br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc \
solid;padding-left:1ex"> <br>
Would we need a serial of the user event (xdg_popup style)? It would<br>
which kinda matches the X event timestamps?<br>
It's trickier as you'd need a protocol to generate a UUID for both<br>
that and the surface to sidechannel, but it's not impossible.<br>
</blockquote>
<br></span>
I don't think that the serial is needed as the compositor would only pass on \
focus if the app which generated the token has currently the focus. Given that the \
serial does not provide any additional advantage. And I don't like just using the \
serial as that could be guessed.<span><br> <br></span></blockquote><div>Just to \
clarify a point. I didn't mean just using the serial as a token. If it was needed \
I meant clients would need to pass seat + serial as args along with the surface to \
our get_token_from_server request.</div><div><br></div><div>but I also didn't \
think it was useful.<br></div><div><br></div><div><br></div><div>> if the app \
which generated the token has currently the focus<br></div><div><br></div><div>We \
might need to tweak that. kickoff will close and lose focus when you click on an \
app before that app is launched.</div><div></div><div>This will probably be something \
up to the compositor, rather than something we need to worry about in the \
spec.</div><div><br></div>-----------</div><div class="gmail_quote"><br></div><div \
class="gmail_quote">X11 startup system is also used for notifying the UI, via taskbar \
or whatever else about the application starting up before it maps a \
surface.</div><div class="gmail_quote"><br></div><div class="gmail_quote"></div><div \
class="gmail_quote">Is that something we want to bring in? (IMHO no)<br></div><div \
class="gmail_quote"><br></div><div class="gmail_quote">David<br></div><div \
class="gmail_quote"><div><br></div></div></div></div>
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic