Merge pull request Expensify#81669 from software-mansion-labs/borys3kk-fix-point-cut-off

Fix points clipping

Author: puneetlath

src/components/Charts/LineChart/LineChartContent.tsx

@@ -25,6 +25,9 @@ const DOT_RADIUS = 6;
 /** Extra hover area beyond the dot radius for easier touch targeting */
 const DOT_HOVER_EXTRA_RADIUS = 2;
 
+/** Minimum safe padding to avoid clipping labels/points */
+const MIN_SAFE_PADDING = DOT_RADIUS + DOT_HOVER_EXTRA_RADIUS;
+
 /** Base domain padding applied to all sides */
 const BASE_DOMAIN_PADDING = {top: 16, bottom: 16, left: 0, right: 0};
 
@@ -96,13 +99,15 @@ function LineChartContent({data, title, titleIcon, isLoading, yAxisUnit, yAxisUn
         // How much space is wasted on each side
         const wastedLeft = geometricPadding - firstLabelNeeds;
         const wastedRight = geometricPadding - lastLabelNeeds;
-        const canReduce = Math.min(wastedLeft, wastedRight);
+        const reclaimablePadding = Math.min(wastedLeft, wastedRight);
 
         // Only reduce if both sides have excess space (labels short enough for 0°)
-        // If canReduce <= 0, labels are too long and hook will use rotation/truncation
-        const horizontalPadding = canReduce > 0 ? geometricPadding - canReduce : geometricPadding;
+        // If reclaimablePadding <= 0, labels are too long and hook will use rotation/truncation
+        const shouldUseExtraPadding = reclaimablePadding > 0;
+        const horizontalPadding = Math.max(shouldUseExtraPadding ? geometricPadding - reclaimablePadding : geometricPadding, MIN_SAFE_PADDING);
 
-        return {...BASE_DOMAIN_PADDING, left: horizontalPadding, right: horizontalPadding};
+        // If shouldUseExtraPadding is true then we have to add the extra padding to the right so the label is not clipped
+        return {...BASE_DOMAIN_PADDING, left: horizontalPadding, right: horizontalPadding + (shouldUseExtraPadding ? MIN_SAFE_PADDING : 0)};
     }, [chartWidth, data, font]);
 
     // For centered labels, tick spacing is evenly distributed across the plot area (same as BarChart)