RFR: 4265389: JSplitPane does not support ComponentOrientation [v5]
Tejesh R
tr at openjdk.org
Fri Jul 19 07:55:32 UTC 2024
On Thu, 18 Jul 2024 13:40:12 GMT, Prasanta Sadhukhan <psadhukhan at openjdk.org> wrote:
>> JSplitPane's support of CompoentOrientation is not present so if orientation is RTL, it still renders left component on left and right component on right instead of other way around..
>> Fix to make it support RTL orientation
>
> Prasanta Sadhukhan has updated the pull request incrementally with one additional commit since the last revision:
>
> Override setComponentOrientation
src/java.desktop/share/classes/javax/swing/JSplitPane.java line 376:
> 374: Component rightComponent = this.getRightComponent();
> 375: if (this.getComponentOrientation().equals(ComponentOrientation.RIGHT_TO_LEFT)) {
> 376: if (rightComponent != null) {
I hope `null` is redundant here, because it is handled inside `setRightComponent/setLeftComponent`. I don't see any difference with and without `null` check while testing in my local. Any particular reason for adding `null` check here?
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/20214#discussion_r1683969100
More information about the client-libs-dev
mailing list