On Fri, 20 Mar 2026 20:40:59 GMT, Marius Hanl <[email protected]> wrote:

>> This PR cleans up the non alternative path of the `MouseEventFirer`. 
>> The code is unused, since the `alternative` mode is always true => the 
>> default.
>> It was also not working correctly, so there really is no need to keep it.
>> 
>> Introduced in: https://github.com/openjdk/jfx/pull/313
>> Default since: https://github.com/openjdk/jfx/pull/852#discussion_r935409172
>
> Marius Hanl has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   Remove now unused scene field

modules/javafx.controls/src/test/java/test/com/sun/javafx/scene/control/infrastructure/MouseEventFirerTest.java
 line 206:

> 204:         // current / alternative mouseEvent creation
> 205:         return List.of(
> 206:             // @Ignore("8253769")

What about https://bugs.openjdk.org/browse/JDK-8253769 - is it still an issue?

-------------

PR Review Comment: https://git.openjdk.org/jfx/pull/2119#discussion_r2967926365

Reply via email to