RFR: 6875229: Wrong placement of icons in JTabbedPane in Nimbus
Jayathirth D V
jdv at openjdk.org
Tue Jul 4 11:39:56 UTC 2023
On Wed, 21 Jun 2023 15:36:14 GMT, Prasanta Sadhukhan <psadhukhan at openjdk.org> wrote:
> Icon is placed after text in JTabbedPane for NimbusL&F since the text is drawn at LEADING position instead of TRAILING as is done for [other L&F](https://github.com/openjdk/jdk/blob/59c6c0e1b7148a7139f5e6a4ba2b19623c20c77d/src/java.desktop/share/classes/javax/swing/plaf/basic/BasicTabbedPaneUI.java#L1155-L1160)
>
> Before fix
> 
>
> After fix
> 
Change looks good to me, except for a small query i have.
Test runs fine with & without fix.
test/jdk/javax/swing/plaf/nimbus/TestNimbusTabbedPaneIconPosition.java line 54:
> 52: static volatile Rectangle bounds;
> 53: private static void addTab(JTabbedPane aTabbedPane, int anIndex) {
> 54: aTabbedPane.addTab("\u2588", new Icon() {
First time i saw usage of Full block character for title and it took some time for me to understand this, unique way to find difference and read color.
-------------
PR Review: https://git.openjdk.org/jdk/pull/14596#pullrequestreview-1512696121
PR Review Comment: https://git.openjdk.org/jdk/pull/14596#discussion_r1251907312
More information about the client-libs-dev
mailing list