RFR: 8317836: FX nodes embedded in JFXPanel need to track component orientation [v6]

Kevin Rushforth kcr at openjdk.org
Sat Oct 28 19:52:41 UTC 2023


On Sat, 28 Oct 2023 18:21:52 GMT, Kevin Rushforth <kcr at openjdk.org> wrote:

>> Prasanta Sadhukhan has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   jcheck
>
> modules/javafx.graphics/src/main/java/javafx/scene/Scene.java line 6352:
> 
>> 6350:                     parent = ((Stage)window).getOwner();
>> 6351:                 } else if (window instanceof com.sun.javafx.embed.EmbeddedSceneInterface) {
>> 6352:                     return ((com.sun.javafx.embed.EmbeddedStageInterface)window).getNodeOrientation();
> 
> This is the right idea, but the wrong type. The window, which is a `javafx.scene.Window` will never be an instance of `EmbeddedSceneInterface` which is a `GlassStage` peer. In the case of `JFXPanel`, the window will be a `com.sun.javafx.stage.EmbeddedWindow`.

I meant "...never an instance of `EmbeddedStageInterface`"

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

PR Review Comment: https://git.openjdk.org/jfx/pull/1271#discussion_r1375314412


More information about the openjfx-dev mailing list