On Tue, 6 Jun 2023 01:09:36 GMT, Thiago Milczarek Sayao <tsa...@openjdk.org> wrote:
>> I replicated the fix on #1054 on Linux. >> >> Also fixes [Monocle: Stage no longer gets focus after fix for >> JDK-8296621](https://bugs.openjdk.org/browse/JDK-8304734) > > Thiago Milczarek Sayao has updated the pull request incrementally with one > additional commit since the last revision: > > Insert copyright I confirm that even restoring [JDK-8296621](https://bugs.openjdk.org/browse/JDK-8296621), the tests were not getting focus. I did this by adding to `MouseEventFirerTest` on `setup`: topLeft.focusedProperty().addListener((observable, oldValue, newValue) -> { System.out.println("topLeft focused " + newValue); }); stage.focusedProperty().addListener((observable, oldValue, newValue) -> { System.out.println("Stage focused " + newValue); }); And adding `testLogging.showStandardStreams = true;` to `build.gradle` With this PR, focus messages are shown, without the PR or restoring [JDK-8296621](https://bugs.openjdk.org/browse/JDK-8296621) they don't. Probably something to do with Monocle flow. ------------- PR Comment: https://git.openjdk.org/jfx/pull/1110#issuecomment-1585775877
