On Tue, 6 Dec 2022 16:46:19 GMT, Michael Strauß wrote:
>> The `Scene.KeyHandler` class doesn't seem to have a clear purpose, mixing
>> focus handling with event propagation. Since #852,
>> `KeyHandler.setFocusVisible` is also called from mouse and touch event
>> handlers, which makes the purpo
On Tue, 20 Dec 2022 20:20:55 GMT, Kevin Rushforth wrote:
> @mstr2 The fix looks fine to me. I presume you have done enough testing to
> ensure no regressions?
>
I tested it by running it against SceneBuilder and playing around with focus,
where I noticed no regressions.
-
PR: ht
On Tue, 20 Dec 2022 16:03:23 GMT, Ambarish Rapte wrote:
>> Michael Strauß has updated the pull request with a new target base due to a
>> merge or a rebase. The pull request now contains three commits:
>>
>> - Merge branch 'master' into fixes/keyhandler-refactor
>> - review changes
>> - Remo
On Tue, 6 Dec 2022 16:46:19 GMT, Michael Strauß wrote:
>> The `Scene.KeyHandler` class doesn't seem to have a clear purpose, mixing
>> focus handling with event propagation. Since #852,
>> `KeyHandler.setFocusVisible` is also called from mouse and touch event
>> handlers, which makes the purpo
On Tue, 6 Dec 2022 16:46:19 GMT, Michael Strauß wrote:
>> The `Scene.KeyHandler` class doesn't seem to have a clear purpose, mixing
>> focus handling with event propagation. Since #852,
>> `KeyHandler.setFocusVisible` is also called from mouse and touch event
>> handlers, which makes the purpo
> The `Scene.KeyHandler` class doesn't seem to have a clear purpose, mixing
> focus handling with event propagation. Since #852,
> `KeyHandler.setFocusVisible` is also called from mouse and touch event
> handlers, which makes the purpose of the class even less pronounced.
>
> Moving the focus-r
> The `Scene.KeyHandler` class doesn't seem to have a clear purpose, mixing
> focus handling with event propagation. Since #852,
> `KeyHandler.setFocusVisible` is also called from mouse and touch event
> handlers, which makes the purpose of the class even less pronounced.
>
> Moving the focus-r
On Thu, 24 Nov 2022 06:49:02 GMT, Michael Strauß wrote:
> The `Scene.KeyHandler` class doesn't seem to have a clear purpose, mixing
> focus handling with event propagation. Since #852,
> `KeyHandler.setFocusVisible` is also called from mouse and touch event
> handlers, which makes the purpose
On Thu, 24 Nov 2022 06:49:02 GMT, Michael Strauß wrote:
> The `Scene.KeyHandler` class doesn't seem to have a clear purpose, mixing
> focus handling with event propagation. Since #852,
> `KeyHandler.setFocusVisible` is also called from mouse and touch event
> handlers, which makes the purpose
The `Scene.KeyHandler` class doesn't seem to have a clear purpose, mixing focus
handling with event propagation. Since #852, `KeyHandler.setFocusVisible` is
also called from mouse and touch event handlers, which makes the purpose of the
class even less pronounced.
Moving the focus-related funct
10 matches
Mail list logo
