fix: fill FlairIcon

launchdarkly · Oct 15, 2024 · 152a5e6 · 152a5e6
1 parent e57a521
commit 152a5e6
Show file tree

Hide file tree

Showing 3 changed files with 20 additions and 5 deletions.
diff --git a/packages/components/src/styles/Button.module.css b/packages/components/src/styles/Button.module.css
@@ -13,10 +13,6 @@
 	border-width: 1px;
 	outline: none;
 	white-space: nowrap;
-
-	& [data-icon] {
-		fill: currentColor;
-	}
 }
 
 .base[data-focus-visible] {
@@ -72,6 +68,10 @@
 	border-color: var(--lp-color-border-interactive-primary-base);
 	color: var(--lp-color-text-interactive-primary-base);
 	z-index: 1;
+
+	& [data-icon] {
+		fill: var(--lp-color-text-interactive-primary-base);
+	}
 }
 
 .primary[data-hovered] {
@@ -97,6 +97,10 @@
 	background-color: var(--lp-color-bg-interactive-destructive-base);
 	border-color: var(--lp-color-border-interactive-destructive);
 	color: var(--lp-color-text-interactive-destructive);
+
+	& [data-icon] {
+		fill: var(--lp-color-fill-interactive-destructive);
+	}
 }
 
 .destructive[data-hovered] {

diff --git a/packages/icons/src/FlairIcon.tsx b/packages/icons/src/FlairIcon.tsx
@@ -1,17 +1,23 @@
 import type { BoxProps } from '@launchpad-ui/box';
 
 import { Box } from '@launchpad-ui/box';
+import { cva, cx } from 'class-variance-authority';
+
+import styles from './styles/FlairIcon.module.css';
+
+const flair = cva(styles.flair);
 
 interface FlairIconProps extends BoxProps {}
 
-const FlairIcon = ({ children, ...props }: FlairIconProps) => {
+const FlairIcon = ({ children, className, ...props }: FlairIconProps) => {
 	return (
 		<Box
 			background="$purple-blue"
 			borderRadius="50%"
 			color="$white.950"
 			display="inline-flex"
 			padding="$400"
+			className={cx(flair(), className)}
 			{...props}
 		>
 			{children}

diff --git a/packages/icons/src/styles/FlairIcon.module.css b/packages/icons/src/styles/FlairIcon.module.css
@@ -0,0 +1,5 @@
+.flair {
+	& [data-icon] {
+		fill: var(--lp-color-white-950);
+	}
+}