From dac63d535547f13d55ffc4de8e4475b7e9f3eeb4 Mon Sep 17 00:00:00 2001
From: pineapplestrikesback <tomasina8@gmail.com>
Date: Sat, 24 Jan 2026 00:32:41 +0100
Subject: [PATCH] fix(chart): remove unwanted borders and overlays from bar
 chart

- Remove browser focus outline on chart elements (yellow border on mobile Safari)
- Remove selection stroke border from selected bars
- Remove hover cursor overlay (blue hue on bars)

Selected bars now indicate selection through color change only (cyan-400 vs cyan-500).

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
---
 src/ui/components/WeeklyActivityChart.tsx | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/src/ui/components/WeeklyActivityChart.tsx b/src/ui/components/WeeklyActivityChart.tsx
index 441c449..a7e30b7 100644
--- a/src/ui/components/WeeklyActivityChart.tsx
+++ b/src/ui/components/WeeklyActivityChart.tsx
@@ -232,7 +232,7 @@ export function WeeklyActivityChart({
       </div>
 
       {/* Chart */}
-      <div className="p-4">
+      <div className="p-4 [&_*]:outline-none [&_*:focus]:outline-none [&_*:focus-visible]:outline-none [&_svg]:outline-none">
         {!hasData ? (
           <EmptyState />
         ) : (
@@ -261,7 +261,7 @@ export function WeeklyActivityChart({
                 domain={[0, maxSets]}
                 allowDataOverflow={false}
               />
-              <Tooltip content={<CustomTooltip />} cursor={{ fill: 'rgba(6, 182, 212, 0.1)' }} />
+              <Tooltip content={<CustomTooltip />} cursor={false} />
               <Bar
                 dataKey="totalSets"
                 radius={[4, 4, 0, 0]}
@@ -289,8 +289,7 @@ export function WeeklyActivityChart({
                     <Cell
                       key={`cell-${index}`}
                       fill={fillColor}
-                      stroke={isSelected ? '#22d3ee' : 'transparent'}
-                      strokeWidth={isSelected ? 2 : 0}
+                      stroke="none"
                     />
                   );
                 })}