From 25426fe7029862914aa7aeb2b6908b6ff8411d0e Mon Sep 17 00:00:00 2001
From: webadderall <131426131+webadderall@users.noreply.github.com>
Date: Mon, 20 Apr 2026 22:03:41 +1000
Subject: [PATCH] revert: restore focus-visible:outline-none on sidebar buttons
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

UI decision — removes default focus ring to avoid ugly yellow selection ring.
---
 src/components/video-editor/EditorSidebar.tsx | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/components/video-editor/EditorSidebar.tsx b/src/components/video-editor/EditorSidebar.tsx
index f2a2de80..2c5ff9fa 100644
--- a/src/components/video-editor/EditorSidebar.tsx
+++ b/src/components/video-editor/EditorSidebar.tsx
@@ -85,11 +85,12 @@ export function EditorSidebar({
 					const isActive = prefs.activeEffectSection === section.id;
 					return (
 						<div key={section.id} className="flex items-center">
+							{/* Intentionally removes default focus-visible ring — UI decision to avoid ugly yellow selection ring */}
 							<motion.button
 								type="button"
 								onClick={() => prefs.setActiveEffectSection(section.id)}
 								title={section.label}
-								className="group relative flex h-9 w-9 items-center justify-center rounded-lg outline-none focus-visible:ring-2 focus-visible:ring-[#2563EB]/50 focus-visible:ring-offset-1"
+								className="group relative flex h-9 w-9 items-center justify-center rounded-lg outline-none focus:outline-none focus-visible:outline-none"
 								animate={{ opacity: isActive ? 1 : 0.55 }}
 								transition={{ duration: 0.14 }}
 							>