fix: Default color for Run and Connect icons (#2448)

* fix: Default color for Run and Connect icons - Default color always has high contrast ration with primary color
microsoft · Nov 25, 2024 · 65f82aa · 65f82aa
1 parent 2dc9ba6
commit 65f82aa
Showing 1 changed file with 2 additions and 9 deletions.
diff --git a/src/webviews/QueryEditor/QueryPanel/QueryToolbar.tsx b/src/webviews/QueryEditor/QueryPanel/QueryToolbar.tsx
@@ -31,15 +31,9 @@ import {
 import { useQueryEditorDispatcher, useQueryEditorState } from '../state/QueryEditorContext';
 
 const useClasses = makeStyles({
-    iconPlay: {
-        color: tokens.colorStatusSuccessBorderActive,
-    },
     iconStop: {
         color: tokens.colorStatusDangerBorderActive,
     },
-    iconConnect: {
-        color: tokens.colorStatusSuccessBorderActive,
-    },
     iconDisconnect: {
         color: tokens.colorStatusDangerBorderActive,
     },
@@ -72,7 +66,7 @@ const BaseActionsSection = () => {
                     {(triggerProps: MenuButtonProps) => (
                         <SplitButton
                             aria-label="Run"
-                            icon={<PlayRegular className={classes.iconPlay} />}
+                            icon={<PlayRegular />}
                             disabled={state.isExecuting || !state.isConnected}
                             appearance={'primary'}
                             menuButton={triggerProps}
@@ -185,14 +179,13 @@ const ConnectedActionsSection = () => {
 };
 
 const DisconnectedActionsSection = () => {
-    const classes = useClasses();
     const dispatcher = useQueryEditorDispatcher();
 
     return (
         <ToolbarButton
             aria-label="Connect"
             appearance={'primary'}
-            icon={<DatabasePlugConnectedRegular className={classes.iconConnect} />}
+            icon={<DatabasePlugConnectedRegular />}
             onClick={() => void dispatcher.connectToDatabase()}
         >
             Connect