From ebdf90c8ab65f7ee6571a9d8aee5a710cea35d30 Mon Sep 17 00:00:00 2001
From: fahed <fahed@softhouse.io>
Date: Tue, 7 Apr 2026 13:11:47 +0300
Subject: [PATCH] fix: use correct translation keys for
 visitors/revenue/bar/pie toggles

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
---
 src/components/Dashboard.tsx | 24 ++++++++++++------------
 src/locales/ar.json          |  6 +++---
 src/locales/en.json          |  6 +++---
 3 files changed, 18 insertions(+), 18 deletions(-)

diff --git a/src/components/Dashboard.tsx b/src/components/Dashboard.tsx
index f2097c0..55f7ec3 100644
--- a/src/components/Dashboard.tsx
+++ b/src/components/Dashboard.tsx
@@ -621,12 +621,12 @@ function Dashboard({ data, seasons, userRole, showDataLabels, setShowDataLabels,
             controls={
               <div style={{ display: 'flex', gap: '6px' }}>
                 <div className="toggle-switch">
-                  <button className={eventMetric === 'visitors' ? 'active' : ''} onClick={() => setEventMetric('visitors')}>{t('dashboard.visitors')}</button>
-                  <button className={eventMetric === 'revenue' ? 'active' : ''} onClick={() => setEventMetric('revenue')}>{t('dashboard.revenue')}</button>
+                  <button className={eventMetric === 'visitors' ? 'active' : ''} onClick={() => setEventMetric('visitors')}>{t('metrics.visitors')}</button>
+                  <button className={eventMetric === 'revenue' ? 'active' : ''} onClick={() => setEventMetric('revenue')}>{t('metrics.revenue')}</button>
                 </div>
                 <div className="toggle-switch">
-                  <button className={eventChartType === 'bar' ? 'active' : ''} onClick={() => setEventChartType('bar')}>{t('presentation.bar')}</button>
-                  <button className={eventChartType === 'pie' ? 'active' : ''} onClick={() => setEventChartType('pie')}>{t('presentation.pie')}</button>
+                  <button className={eventChartType === 'bar' ? 'active' : ''} onClick={() => setEventChartType('bar')}>{t('metrics.bar')}</button>
+                  <button className={eventChartType === 'pie' ? 'active' : ''} onClick={() => setEventChartType('pie')}>{t('metrics.pie')}</button>
                 </div>
               </div>
             }
@@ -651,8 +651,8 @@ function Dashboard({ data, seasons, userRole, showDataLabels, setShowDataLabels,
             className="chart-container"
             controls={
               <div className="toggle-switch">
-                <button className={channelChartType === 'bar' ? 'active' : ''} onClick={() => setChannelChartType('bar')}>{t('presentation.bar')}</button>
-                <button className={channelChartType === 'pie' ? 'active' : ''} onClick={() => setChannelChartType('pie')}>{t('presentation.pie')}</button>
+                <button className={channelChartType === 'bar' ? 'active' : ''} onClick={() => setChannelChartType('bar')}>{t('metrics.bar')}</button>
+                <button className={channelChartType === 'pie' ? 'active' : ''} onClick={() => setChannelChartType('pie')}>{t('metrics.pie')}</button>
               </div>
             }
           >
@@ -738,12 +738,12 @@ function Dashboard({ data, seasons, userRole, showDataLabels, setShowDataLabels,
                   <h2>{eventMetric === 'visitors' ? t('dashboard.visitorsByMuseum') : t('dashboard.revenueByMuseum')}</h2>
                   <div style={{ display: 'flex', gap: '6px', marginBottom: '8px' }}>
                     <div className="toggle-switch">
-                      <button className={eventMetric === 'visitors' ? 'active' : ''} onClick={() => setEventMetric('visitors')}>{t('dashboard.visitors')}</button>
-                      <button className={eventMetric === 'revenue' ? 'active' : ''} onClick={() => setEventMetric('revenue')}>{t('dashboard.revenue')}</button>
+                      <button className={eventMetric === 'visitors' ? 'active' : ''} onClick={() => setEventMetric('visitors')}>{t('metrics.visitors')}</button>
+                      <button className={eventMetric === 'revenue' ? 'active' : ''} onClick={() => setEventMetric('revenue')}>{t('metrics.revenue')}</button>
                     </div>
                     <div className="toggle-switch">
-                      <button className={eventChartType === 'bar' ? 'active' : ''} onClick={() => setEventChartType('bar')}>{t('presentation.bar')}</button>
-                      <button className={eventChartType === 'pie' ? 'active' : ''} onClick={() => setEventChartType('pie')}>{t('presentation.pie')}</button>
+                      <button className={eventChartType === 'bar' ? 'active' : ''} onClick={() => setEventChartType('bar')}>{t('metrics.bar')}</button>
+                      <button className={eventChartType === 'pie' ? 'active' : ''} onClick={() => setEventChartType('pie')}>{t('metrics.pie')}</button>
                     </div>
                   </div>
                   <div className="chart-container">
@@ -768,8 +768,8 @@ function Dashboard({ data, seasons, userRole, showDataLabels, setShowDataLabels,
                 <div className="chart-card">
                   <h2>{t('dashboard.channelPerformance')}</h2>
                   <div className="toggle-switch" style={{ marginBottom: '8px' }}>
-                    <button className={channelChartType === 'bar' ? 'active' : ''} onClick={() => setChannelChartType('bar')}>{t('presentation.bar')}</button>
-                    <button className={channelChartType === 'pie' ? 'active' : ''} onClick={() => setChannelChartType('pie')}>{t('presentation.pie')}</button>
+                    <button className={channelChartType === 'bar' ? 'active' : ''} onClick={() => setChannelChartType('bar')}>{t('metrics.bar')}</button>
+                    <button className={channelChartType === 'pie' ? 'active' : ''} onClick={() => setChannelChartType('pie')}>{t('metrics.pie')}</button>
                   </div>
                   <div className="chart-container">
                     {channelChartType === 'bar'
diff --git a/src/locales/ar.json b/src/locales/ar.json
index a7cdd0c..a9a7d3c 100644
--- a/src/locales/ar.json
+++ b/src/locales/ar.json
@@ -57,7 +57,9 @@
     "avgRevenue": "متوسط الإيراد/زائر",
     "avgRevenuePerVisitor": "متوسط الإيراد لكل زائر",
     "pilgrims": "المعتمرون",
-    "captureRate": "نسبة الاستقطاب"
+    "captureRate": "نسبة الاستقطاب",
+    "bar": "أعمدة",
+    "pie": "دائري"
   },
   "dashboard": {
     "title": "لوحة التحكم",
@@ -137,8 +139,6 @@
     "addFirst": "أضف شريحتك الأولى",
     "slideTitle": "عنوان الشريحة",
     "chartType": "نوع الرسم البياني",
-    "bar": "أعمدة",
-    "pie": "دائري",
     "metric": "المقياس",
     "startDate": "تاريخ البداية",
     "endDate": "تاريخ النهاية",
diff --git a/src/locales/en.json b/src/locales/en.json
index 3281a78..fd47ff0 100644
--- a/src/locales/en.json
+++ b/src/locales/en.json
@@ -57,7 +57,9 @@
     "avgRevenue": "Avg Rev/Visitor",
     "avgRevenuePerVisitor": "Avg Revenue/Visitor",
     "pilgrims": "Pilgrims",
-    "captureRate": "Capture Rate"
+    "captureRate": "Capture Rate",
+    "bar": "Bar",
+    "pie": "Pie"
   },
   "dashboard": {
     "title": "Dashboard",
@@ -137,8 +139,6 @@
     "addFirst": "Add your first slide",
     "slideTitle": "Slide Title",
     "chartType": "Chart Type",
-    "bar": "Bar",
-    "pie": "Pie",
     "metric": "Metric",
     "startDate": "Start Date",
     "endDate": "End Date",