Table: optimized position of column sort arrow (issue #34)

JFormDesigner · Feb 8, 2020 · 686d667 · 686d667
1 parent 26d603d
commit 686d667
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 3 deletions.
diff --git a/flatlaf-core/src/main/java/com/formdev/flatlaf/icons/FlatAscendingSortIcon.java b/flatlaf-core/src/main/java/com/formdev/flatlaf/icons/FlatAscendingSortIcon.java
@@ -47,7 +47,7 @@ protected void paintIcon( Component c, Graphics2D g ) {
 		g.setColor( sortIconColor );
 		if( chevron ) {
 			// chevron arrow
-			Path2D path = FlatUIUtils.createPath( false, 1,5, 5,1, 9,5 );
+			Path2D path = FlatUIUtils.createPath( false, 1,4, 5,0, 9,4 );
 			g.setStroke( new BasicStroke( 1f ) );
 			g.draw( path );
 		} else {

diff --git a/flatlaf-core/src/main/java/com/formdev/flatlaf/icons/FlatDescendingSortIcon.java b/flatlaf-core/src/main/java/com/formdev/flatlaf/icons/FlatDescendingSortIcon.java
@@ -47,7 +47,7 @@ protected void paintIcon( Component c, Graphics2D g ) {
 		g.setColor( sortIconColor );
 		if( chevron ) {
 			// chevron arrow
-			Path2D path = FlatUIUtils.createPath( false, 1,1, 5,5, 9,1 );
+			Path2D path = FlatUIUtils.createPath( false, 1,0, 5,4, 9,0 );
 			g.setStroke( new BasicStroke( 1f ) );
 			g.draw( path );
 		} else {

diff --git a/flatlaf-core/src/main/java/com/formdev/flatlaf/ui/FlatTableHeaderUI.java b/flatlaf-core/src/main/java/com/formdev/flatlaf/ui/FlatTableHeaderUI.java
@@ -307,7 +307,9 @@ public void paintBorder( Component c, Graphics g, int x, int y, int width, int h
 				int xi = x + ((width - sortIcon.getIconWidth()) / 2);
 				int yi = (sortIconPosition == SwingConstants.TOP)
 					? y + UIScale.scale( 1 )
-					: y + height - sortIcon.getIconHeight() - UIScale.scale( 1 );
+					: y + height - sortIcon.getIconHeight()
+						- 1 // for gap
+						- (int) (1 * UIScale.getUserScaleFactor()); // for bottom border
 				sortIcon.paintIcon( c, g, xi, yi );
 			}
 		}