RFR: 8306121: Scene not rendered initially when changing scenes after fix for JDK-8296621 [v20]

Thiago Milczarek Sayao tsayao at openjdk.org
Sat Jun 10 19:15:56 UTC 2023


On Tue, 6 Jun 2023 01:09:36 GMT, Thiago Milczarek Sayao <tsayao at 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


More information about the openjfx-dev mailing list