Re: [PATCH weston 2/2] Reset the keyboard focus on all seats when the compositor is locked

2014-04-10 Thread Kristian Høgsberg
On Wed, Apr 09, 2014 at 04:33:32PM +0100, Neil Roberts wrote: > Before commit 2f5faff7f9142 when the compositor is locked it would > reset the keyboard focus on all of the seats as part of pushing the > focus_state. This was removed because it now always keeps track of the > focus_state in the work

Re: [PATCH weston 2/2] Reset the keyboard focus on all seats when the compositor is locked

2014-04-10 Thread Pekka Paalanen
On Wed, 09 Apr 2014 17:08:23 +0100 Neil Roberts wrote: > "Jasper St. Pierre" writes: > > > Why is this necessary? Won't events never be delivered while locked > > anyway? > > Events do seem to get delivered, you can try it if you run weston -i5 > and then open a terminal and let it lock. You c

Re: [PATCH weston 2/2] Reset the keyboard focus on all seats when the compositor is locked

2014-04-09 Thread Neil Roberts
"Jasper St. Pierre" writes: > Why is this necessary? Won't events never be delivered while locked > anyway? Events do seem to get delivered, you can try it if you run weston -i5 and then open a terminal and let it lock. You can still type in the terminal and launch programs. But yeah, I guess a

Re: [PATCH weston 2/2] Reset the keyboard focus on all seats when the compositor is locked

2014-04-09 Thread Jasper St. Pierre
Why is this necessary? Won't events never be delivered while locked anyway? On Apr 9, 2014 8:36 AM, "Neil Roberts" wrote: > Before commit 2f5faff7f9142 when the compositor is locked it would > reset the keyboard focus on all of the seats as part of pushing the > focus_state. This was removed beca

[PATCH weston 2/2] Reset the keyboard focus on all seats when the compositor is locked

2014-04-09 Thread Neil Roberts
Before commit 2f5faff7f9142 when the compositor is locked it would reset the keyboard focus on all of the seats as part of pushing the focus_state. This was removed because it now always keeps track of the focus_state in the workspace instead of waiting until the state is pushed. However this had t