RFR: 8306121: Scene not rendered initially when changing scenes after fix for JDK-8296621 [v21]
Kevin Rushforth
kcr at openjdk.org
Thu Jun 15 12:56:17 UTC 2023
On Thu, 15 Jun 2023 12:16:20 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:
>
> Review changes
Copyright year changes are incorrect.
modules/javafx.controls/src/test/java/test/com/sun/javafx/scene/control/infrastructure/MouseEventFirerTest.java line 2:
> 1: /*
> 2: * Copyright (c) 2023, Oracle and/or its affiliates. All rights reserved.
No, we never remove the start year in a copyright line. this should be: `2020, 2023,`
Alternatively, you can just revert this change, since we will run a script to update them all. Either is fine.
modules/javafx.controls/src/test/java/test/javafx/scene/control/ComboBoxTest.java line 2:
> 1: /*
> 2: * Copyright (c) 2011, 2023, Oracle and/or its affiliates. All rights reserved.
`2022, 2023,`
-------------
Changes requested by kcr (Lead).
PR Review: https://git.openjdk.org/jfx/pull/1110#pullrequestreview-1481491145
PR Review Comment: https://git.openjdk.org/jfx/pull/1110#discussion_r1230961089
PR Review Comment: https://git.openjdk.org/jfx/pull/1110#discussion_r1230961274
More information about the openjfx-dev
mailing list