On Tue, 31 Oct 2023 16:13:57 GMT, Prasanta Sadhukhan <psadhuk...@openjdk.org> 
wrote:

>> FX Nodes embedded in a Swing JFXPanel does not track the component 
>> orientation and FX nodes remain unaffected when component orientation 
>> changes.
>> Fix made sure JavaFX scene embedded in a JFXPanel should inherit the value 
>> from the JFXPanel.
>
> Prasanta Sadhukhan has updated the pull request incrementally with one 
> additional commit since the last revision:
> 
>   Treat UNKNOWN as LTR

> > For the Scene, yes, but we aren't talking about that.
> 
> I got confused by the variable name "stage". You are right!
> 
> The last code 
> [3b7be5d](https://github.com/openjdk/jfx/commit/3b7be5d9888a24a5d1db2a5abcad3c647533544f)
>  works as expected.
> 
> Perhaps we can clean up the logic a bit, something like
> 
> ```
> boolean rtl = ComponentOrientation.RIGHT_TO_LEFT.equals(cor);
> stage.setNodeOrientation(rtl ? RTL : LTR)
> ```

Sure..done..

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

PR Comment: https://git.openjdk.org/jfx/pull/1271#issuecomment-1787574852

Reply via email to