fix(dashboard): restore ResponsiveContainer in ChartContainer (#535)

The dashboard's ChartContainer was missing Recharts' ResponsiveContainer
wrapper and the aspect-video CSS class, causing charts to render with no
dimensions. The ChartWrapper was using a cloneElement hack to compensate,
but this broke after tooltip sync changes.

Restore ResponsiveContainer and aspect-video to match the OSS
implementation, and remove the now-unnecessary cloneElement workaround.

Author: GenerQAQ

dashboard/app/project/[id]/dashboard-charts.tsx

@@ -1,6 +1,6 @@
 "use client";
 
-import React, { memo, useMemo, cloneElement } from "react";
+import React, { memo, useMemo } from "react";
 import { Loader2 } from "lucide-react";
 import { Card, CardContent } from "@/components/ui/card";
 import {
@@ -346,15 +346,9 @@ export const ChartWrapper = memo(function ChartWrapper({
     );
   }
 
-  // Clone the chart element and inject responsive prop and style
-  const chartWithResponsive = cloneElement(children as React.ReactElement<{ responsive?: boolean; style?: React.CSSProperties }>, {
-    responsive: true,
-    style: { width: "100%", height: "100%" },
-  });
-
   return (
     <ChartContainer config={chartConfig} className="h-full w-full">
-      {chartWithResponsive}
+      {children}
     </ChartContainer>
   );
 });

dashboard/components/ui/chart.tsx

@@ -80,7 +80,9 @@ function ChartContainer({
   ...props
 }: React.ComponentProps<"div"> & {
   config: ChartConfig
-  children: React.ReactNode
+  children: React.ComponentProps<
+    typeof RechartsPrimitive.ResponsiveContainer
+  >["children"]
 }) {
   const uniqueId = React.useId()
   const chartId = `chart-${id || uniqueId.replace(/:/g, "")}`
@@ -91,13 +93,15 @@ function ChartContainer({
         data-slot="chart"
         data-chart={chartId}
         className={cn(
-          "[&_.recharts-cartesian-axis-tick_text]:fill-muted-foreground [&_.recharts-cartesian-grid_line[stroke='#ccc']]:stroke-border/50 [&_.recharts-curve.recharts-tooltip-cursor]:stroke-border [&_.recharts-polar-grid_[stroke='#ccc']]:stroke-border [&_.recharts-radial-bar-background-sector]:fill-muted [&_.recharts-rectangle.recharts-tooltip-cursor]:fill-muted [&_.recharts-reference-line_[stroke='#ccc']]:stroke-border flex justify-center text-xs [&_.recharts-dot[stroke='#fff']]:stroke-transparent [&_.recharts-layer]:outline-hidden [&_.recharts-sector]:outline-hidden [&_.recharts-sector[stroke='#fff']]:stroke-transparent [&_.recharts-surface]:outline-hidden",
+          "[&_.recharts-cartesian-axis-tick_text]:fill-muted-foreground [&_.recharts-cartesian-grid_line[stroke='#ccc']]:stroke-border/50 [&_.recharts-curve.recharts-tooltip-cursor]:stroke-border [&_.recharts-polar-grid_[stroke='#ccc']]:stroke-border [&_.recharts-radial-bar-background-sector]:fill-muted [&_.recharts-rectangle.recharts-tooltip-cursor]:fill-muted [&_.recharts-reference-line_[stroke='#ccc']]:stroke-border flex aspect-video justify-center text-xs [&_.recharts-dot[stroke='#fff']]:stroke-transparent [&_.recharts-layer]:outline-hidden [&_.recharts-sector]:outline-hidden [&_.recharts-sector[stroke='#fff']]:stroke-transparent [&_.recharts-surface]:outline-hidden",
           className
         )}
         {...props}
       >
         <ChartStyle id={chartId} config={config} />
-        {children}
+        <RechartsPrimitive.ResponsiveContainer>
+          {children}
+        </RechartsPrimitive.ResponsiveContainer>
       </div>
     </ChartContext.Provider>
   )