From 9e20dc997afa2f44869f897ed5c8efb7cf0b9119 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Cau=C3=AA=20Faleiros?= <cauefaleiros@gmail.com>
Date: Wed, 6 May 2026 18:23:53 -0300
Subject: [PATCH] feat: implement consistent 20-color mapping for visible
 products across dashboard charts

---
 src/pages/Dashboard.tsx | 42 +++++++++++++++++++++--------------------
 1 file changed, 22 insertions(+), 20 deletions(-)

diff --git a/src/pages/Dashboard.tsx b/src/pages/Dashboard.tsx
index ae95e3c..159be65 100644
--- a/src/pages/Dashboard.tsx
+++ b/src/pages/Dashboard.tsx
@@ -7,8 +7,12 @@ import type { OrderData, DateRange } from '../types';
 import { parseOrderDate } from '../dataService';
 
 const COLORS = [
+  // 10 Strong Base Colors
   '#10b981', '#3b82f6', '#8b5cf6', '#f43f5e', '#f97316', 
-  '#06b6d4', '#ec4899', '#eab308', '#6366f1', '#14b8a6'
+  '#06b6d4', '#ec4899', '#eab308', '#6366f1', '#14b8a6',
+  // 10 Softer Versions
+  '#6ee7b7', '#93c5fd', '#c4b5fd', '#fda4af', '#fdba74',
+  '#67e8f9', '#f9a8d4', '#fde047', '#a5b4fc', '#5eead4'
 ];
 
 const formatCurrency = (value: number) => {
@@ -63,35 +67,33 @@ const Dashboard = () => {
       }
     });
 
-    const uniqueProducts = Array.from(new Set(filteredData.map(order => order.Descricao_Produto.split(' TAMANHO')[0]))).sort();
+    // Identify which products will actually be displayed in both charts (Top 10 of each)
+    const topSalesNames = Object.keys(productSalesMap).sort((a, b) => productSalesMap[b] - productSalesMap[a]).slice(0, 10);
+    const topRevenueNames = Object.keys(productRevenueMap).sort((a, b) => productRevenueMap[b] - productRevenueMap[a]).slice(0, 10);
+    
+    // Combine them into a unique set to assign colors only to the VISIBLE products
+    const displayProducts = Array.from(new Set([...topSalesNames, ...topRevenueNames])).sort();
     const productColors: Record<string, string> = {};
-    uniqueProducts.forEach((productName, index) => {
-      productColors[productName] = COLORS[index % COLORS.length];
+    
+    displayProducts.forEach((name, index) => {
+      productColors[name] = COLORS[index % COLORS.length];
     });
 
-    const productsData = Object.keys(productSalesMap).map(key => ({
-      name: key,
-      value: productSalesMap[key]
-    })).sort((a, b) => b.value - a.value).slice(0, 10).map(item => ({
-      ...item,
-      fill: productColors[item.name]
+    const productsData = topSalesNames.map(name => ({
+      name,
+      value: productSalesMap[name],
+      fill: productColors[name]
     }));
 
-    const revenueData = Object.keys(productRevenueMap).map(key => ({
-      name: key,
-      value: productRevenueMap[key]
-    })).sort((a, b) => b.value - a.value).slice(0, 10).map(item => ({
-      ...item,
-      fill: productColors[item.name]
+    const revenueData = topRevenueNames.map(name => ({
+      name,
+      value: productRevenueMap[name],
+      fill: productColors[name]
     }));
 
     return { totalRevenue: revenue, totalOrders: totalItems, averageOrderValue: revenue / (filteredData.length || 1), salesByProduct: productsData, revenueByProduct: revenueData };
   }, [filteredData]);
 
-  const formatCurrency = (value: number) => {
-    return new Intl.NumberFormat('pt-BR', { style: 'currency', currency: 'BRL' }).format(value);
-  };
-
   return (
     <div className="space-y-6">
       <div className="flex flex-col md:flex-row md:items-center justify-between gap-4">