[@mantine/charts] Sparkline: Fix incorrect data prop type (#6352)

mantinedev · Jul 12, 2024 · 392880f · 392880f
1 parent 308761e
commit 392880f
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/packages/@mantine/charts/src/Sparkline/Sparkline.tsx b/packages/@mantine/charts/src/Sparkline/Sparkline.tsx
@@ -33,7 +33,7 @@ export interface SparklineProps
     StylesApiProps<SparklineFactory>,
     ElementProps<'div'> {
   /** Data used to render the chart */
-  data: number[];
+  data: (number | null)[];
 
   /** Key of `theme.colors` or any valid CSS color, `theme.primaryColor` by default */
   color?: MantineColor;
@@ -75,10 +75,14 @@ const defaultProps: Partial<SparklineProps> = {
   curveType: 'linear',
 };
 
-function getTrendColor(data: number[], trendColors: SparklineTrendColors) {
+function getTrendColor(data: (number | null)[], trendColors: SparklineTrendColors) {
   const first = data[0];
   const last = data[data.length - 1];
 
+  if (first === null || last === null) {
+    return trendColors.neutral || trendColors.positive;
+  }
+
   if (first < last) {
     return trendColors.positive;
   }