[PATCH 2/2] protocol: add state set functions for maximized and fullscreen.
Jason Ekstrand
jason at jlekstrand.net
Sat Nov 2 16:50:40 CET 2013
On Fri, Nov 1, 2013 at 11:36 PM, Jasper St. Pierre <jstpierre at mecheye.net>wrote:
> On Fri, Nov 1, 2013 at 11:58 PM, Jason Ekstrand <jason at jlekstrand.net>wrote:
>
> ... snip
>
>
>> Gregory,
>> Thank you very much for your e-mail. I think that helps a lot. The lack
>> of code is ok because I think Rafael is planning to start implementing as
>> soon as things settle a bit. Sometimes protocol discussions can end up
>> with a whole lot of hypotheticals and what you gave was a very clear
>> concise discussion of the topic.
>>
>> If I am understanding you correctly, what we need are an "activate"
>> request and "notify_active" and "notify_inactive" events. To support
>> sloppy focus, clients should just always try to activate on
>> wl_pointer.enter and let the compositor sort it out unless they have a good
>> reason to do otherwise. For cases such as alt-tab, or another window
>> closing, the compositor simply sends a notify_active. I think I'm ok with
>> this.
>>
>> Two more questions that come to mind:
>>
>> 1) Will we ever need a deactivate request? Under the given scheme, no
>> need comes immediately to mind.
>>
>
> No. Well, if we need it, let's add it later.
>
> 2) Should the activate be associated with a particular seat. If you have
>> multiple cursors, you can easily have multiple active windows so this seems
>> perfectly reasonable. If this is the case, should this be part of wl_seat
>> or should we keep it in xdg_shell? I'm a little afraid to clutter wl_seat
>> unnecessarily but this is very quickly starting to look like a seat focus.
>>
>
> I don't think windows need to know which seat they're active on, only that
> they're active on possibly one of them. From my eyes, there's nothing that
> prevents the compositor to send "notify_active" to more than one window at
> the same time for multiseat support.
>
> If I click on an unresponsive window A, we'll send a "click" event to that
> window, but since it's unresponsive, it won't activate. I might then decide
> to activate another window B, which will respond. When the window A becomes
> responsive again, it will process the "click" event and decide to send an
> "activate" request back. We need to make sure that this "rogue" client
> doesn't steal the focus, so the compositor needs to record the serial for
> the last "activate" event it got (sent by window B), and compare against it
> when it gets an "activate" event, besides whatever policies it might have.
>
Here is the problem. Say you have two pointers P and Q and two and two
windows A and B. P clicks on A and A a is now active. Q clicks on B and
now B is active. Then P goes and clicks on B. However, B is already
active so it doesn't request an activation and A stays active with pointer
P. This assumes we are trusting clients to request activation. The other
option is that we simply force a click-always-activates model and clients
are constantly fighting the compositor just to make modal dialogues work.
>
> But serials are seat-specific, so we need the "activate" request to relay
> the seat back for the purposes of validating the timestamp. I don't think
> it makes sense to expose the active window on wl_seat, though.
>
That's not correct. Timestamps are seat specific but timestamps are not
the same as serials. Timestamps are local to the seat. Serials are
actually compositor-global in weston and, in order to make things work,
should probably at least be client-global in other compositors.
>
> Once again, Gregory, Thanks for the explanation. I hope I'm following ok.
>> --Jason Ekstrand
>>
>> _______________________________________________
>> wayland-devel mailing list
>> wayland-devel at lists.freedesktop.org
>> http://lists.freedesktop.org/mailman/listinfo/wayland-devel
>>
>>
>
>
> --
> Jasper
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/wayland-devel/attachments/20131102/3ab8430d/attachment-0001.html>
More information about the wayland-devel
mailing list