fix: apply primary button styles for icon button #2121

h2oai · Sep 13, 2023 · c5a93a2 · c5a93a2
1 parent 19021ae
commit c5a93a2
Showing 1 changed file with 21 additions and 2 deletions.
diff --git a/ui/src/button.tsx b/ui/src/button.tsx
@@ -115,6 +115,11 @@ const
     center: 'center',
     between: 'space-between',
     around: 'space-around',
+  },
+  iconButtonPrimaryStyles = {
+    root: { color: cssVar('$white'), background: cssVar('$themePrimary') },
+    rootHovered: { color: cssVar('$white'), background: cssVar('$themeDarkAlt') },
+    rootPressed: { color: cssVar('$white'), background: cssVar('$themePrimary') }
   }
 
 const
@@ -171,8 +176,22 @@ const
       } : undefined,
       split: !!commands,
     }
-    if (isIconOnly) return <Fluent.IconButton {...btnProps} data-test={name} title={caption} />
-
+    if (isIconOnly) {
+      return <Fluent.IconButton
+        {...btnProps}
+        data-test={name}
+        title={caption}
+        // Fluent does not support primary icon buttons so we need to override the styles.
+        styles={primary
+          ? {
+            ...styles,
+            ...iconButtonPrimaryStyles,
+            root: { ...(styles.root as any), ...iconButtonPrimaryStyles.root }
+          }
+          : undefined
+        }
+      />
+    }
     return caption?.length
       ? primary
         ? <Fluent.CompoundButton {...btnProps} data-test={name} primary secondaryText={caption} />