fix: label position fix (#1357)

fixes #1266
chartist-js · Sep 23, 2022 · fbc13e2 · fbc13e2
1 parent b90cfea
commit fbc13e2
Show file tree

Hide file tree

Showing 6 changed files with 4 additions and 1 deletion.
diff --git a/src/axes/Axis.ts b/src/axes/Axis.ts
@@ -91,7 +91,10 @@ export abstract class Axis {
         // If we don't have a label ahead and we have only two labels in total, we just take the remaining distance to
         // on the whole axis length. We limit that to a minimum of 30 pixel, so that labels close to the border will
         // still be visible inside of the chart padding.
-        labelLength = Math.max(this.axisLength - projectedValue, 30);
+        labelLength = Math.max(
+          this.axisLength - projectedValue,
+          this.axisLength / this.ticks.length
+        );
       }
 
       // Skip grid lines and labels where interpolated label values are falsey (except for 0)

diff --git a/test/__image_snapshots__/BarChart__Adaptive__iPad-snap.png b/test/__image_snapshots__/BarChart__Adaptive__iPad-snap.png
diff --git a/test/__image_snapshots__/BarChart__Adaptive__iPhone-X-landscape-snap.png b/test/__image_snapshots__/BarChart__Adaptive__iPhone-X-landscape-snap.png
diff --git a/test/__image_snapshots__/BarChart__Default-snap.png b/test/__image_snapshots__/BarChart__Default-snap.png
diff --git a/test/__image_snapshots__/BarChart__Horizontal-snap.png b/test/__image_snapshots__/BarChart__Horizontal-snap.png
diff --git a/test/__image_snapshots__/BarChart__Overlapping-Bars-On-Mobile__iPhone-X-snap.png b/test/__image_snapshots__/BarChart__Overlapping-Bars-On-Mobile__iPhone-X-snap.png