refinenments

apps/mobile/src/components/WeeklyProgressChart.tsx

@@ -1,7 +1,7 @@
 import React from "react";
 import { View, Text, StyleSheet, Dimensions } from "react-native";
 import { LineChart } from "react-native-chart-kit";
-import { theme } from "../styles/theme";
+import { useTheme } from "../contexts/ThemeContext";
 import type { WeeklyTrendData } from "../api/types";
 
 interface WeeklyProgressChartProps {
@@ -13,28 +13,26 @@ export function WeeklyProgressChart({
   weeklyData,
   title = "Weekly Progress",
 }: WeeklyProgressChartProps) {
+  const { colors, typography } = useTheme();
   const screenWidth = Dimensions.get("window").width;
 
-  // Prepare chart data
-  const labels = weeklyData.map((week) => week.weekLabel);
-  const checkInsData = weeklyData.map((week) => week.checkIns);
-  const goalsCompletedData = weeklyData.map((week) => week.goalsCompleted);
-  const avgProgressData = weeklyData.map((week) => week.averageProgress);
+  const labels = weeklyData.map((week: WeeklyTrendData) => week.weekLabel);
+  const checkInsData = weeklyData.map((week: WeeklyTrendData) => week.checkIns);
+  const goalsCompletedData = weeklyData.map(
+    (week: WeeklyTrendData) => week.goalsCompleted,
+  );
 
   const chartConfig = {
-    backgroundColor: theme.colors.white,
-    backgroundGradientFrom: theme.colors.white,
-    backgroundGradientTo: theme.colors.white,
+    backgroundColor: colors.surface,
+    backgroundGradientFrom: colors.surface,
+    backgroundGradientTo: colors.surface,
     decimalPlaces: 0,
-    color: (opacity = 1) => `rgba(59, 130, 246, ${opacity})`,
-    labelColor: (opacity = 1) => `rgba(107, 114, 128, ${opacity})`,
-    style: {
-      borderRadius: theme.borderRadius.lg,
-    },
+    color: (opacity = 1) => `rgba(0, 102, 255, ${opacity})`,
+    labelColor: (opacity = 1) => colors.textTertiary,
     propsForDots: {
-      r: "4",
+      r: "5",
       strokeWidth: "2",
-      stroke: theme.colors.primary,
+      stroke: colors.primary,
     },
   };
 
@@ -43,31 +41,40 @@ export function WeeklyProgressChart({
     datasets: [
       {
         data: checkInsData,
-        color: (opacity = 1) => `rgba(59, 130, 246, ${opacity})`, // Blue for check-ins
-        strokeWidth: 2,
+        color: () => colors.primary,
+        strokeWidth: 3,
       },
       {
         data: goalsCompletedData,
-        color: (opacity = 1) => `rgba(16, 185, 129, ${opacity})`, // Green for goals
-        strokeWidth: 2,
+        color: () => colors.success,
+        strokeWidth: 3,
       },
     ],
-    legend: ["Check-ins", "Goals Completed"],
+    legend: ["Check-ins", "Goals"],
   };
 
   return (
     <View style={styles.container}>
-      <Text style={styles.title}>{title}</Text>
+      {title && (
+        <Text
+          style={[
+            typography.h4,
+            { color: colors.textPrimary, marginBottom: 16 },
+          ]}
+        >
+          {title}
+        </Text>
+      )}
       <View style={styles.chartContainer}>
         <LineChart
           data={data}
-          width={screenWidth - 60}
-          height={220}
+          width={screenWidth - 80}
+          height={180}
           chartConfig={chartConfig}
           bezier
           style={styles.chart}
           withInnerLines={true}
-          withOuterLines={true}
+          withOuterLines={false}
           withVerticalLabels={true}
           withHorizontalLabels={true}
           fromZero={true}
@@ -75,12 +82,20 @@ export function WeeklyProgressChart({
       </View>
       <View style={styles.legend}>
         <View style={styles.legendItem}>
-          <View style={[styles.legendDot, { backgroundColor: "#3b82f6" }]} />
-          <Text style={styles.legendText}>Check-ins</Text>
+          <View
+            style={[styles.legendDot, { backgroundColor: colors.primary }]}
+          />
+          <Text style={[typography.caption, { color: colors.textSecondary }]}>
+            Check-ins
+          </Text>
         </View>
         <View style={styles.legendItem}>
-          <View style={[styles.legendDot, { backgroundColor: "#10b981" }]} />
-          <Text style={styles.legendText}>Goals Completed</Text>
+          <View
+            style={[styles.legendDot, { backgroundColor: colors.success }]}
+          />
+          <Text style={[typography.caption, { color: colors.textSecondary }]}>
+            Goals
+          </Text>
         </View>
       </View>
     </View>
@@ -89,17 +104,8 @@ export function WeeklyProgressChart({
 
 const styles = StyleSheet.create({
   container: {
-    backgroundColor: theme.colors.white,
-    borderRadius: theme.borderRadius.xl,
-    padding: 16,
-    marginBottom: 16,
-    ...theme.shadows.medium,
-  },
-  title: {
-    fontSize: theme.typography.fontSize.lg,
-    fontWeight: theme.typography.fontWeight.bold,
-    color: theme.colors.gray700,
-    marginBottom: 12,
+    borderRadius: 16,
+    padding: 4,
   },
   chartContainer: {
     alignItems: "center",
@@ -107,26 +113,22 @@ const styles = StyleSheet.create({
   },
   chart: {
     marginVertical: 8,
-    borderRadius: theme.borderRadius.lg,
+    borderRadius: 12,
   },
   legend: {
     flexDirection: "row",
     justifyContent: "center",
-    gap: 20,
+    gap: 24,
     paddingTop: 8,
   },
   legendItem: {
     flexDirection: "row",
     alignItems: "center",
-    gap: 6,
+    gap: 8,
   },
   legendDot: {
     width: 10,
     height: 10,
     borderRadius: 5,
   },
-  legendText: {
-    fontSize: theme.typography.fontSize.sm,
-    color: theme.colors.gray500,
-  },
 });