From 63fcac7aeb917ded4eb408ff6b5a8d594ec9059f Mon Sep 17 00:00:00 2001
From: hannesrudolph <hrudolph@gmail.com>
Date: Mon, 28 Jul 2025 18:15:53 -0600
Subject: [PATCH] fix: correct logical operator in export button className

- Change OR (||) to AND (&&) operator for conditional opacity classes
- Fixes issue where string operand would always be truthy
- Addresses additional Copilot review feedback
---
 webview-ui/src/components/chat/ModeSelector.tsx | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/webview-ui/src/components/chat/ModeSelector.tsx b/webview-ui/src/components/chat/ModeSelector.tsx
index 51211d39d0..7a0cb04083 100644
--- a/webview-ui/src/components/chat/ModeSelector.tsx
+++ b/webview-ui/src/components/chat/ModeSelector.tsx
@@ -311,7 +311,7 @@ export const ModeSelector = ({
 													size="icon"
 													className={cn(
 														"h-6 w-6 p-0.5",
-														isExporting === mode.slug ||
+														isExporting !== mode.slug &&
 															"opacity-0 group-hover:opacity-100",
 														"transition-opacity",
 													)}