From b26e0469be61165f6541a82b3e7806968e3308dd Mon Sep 17 00:00:00 2001 From: Thomas Heartman Date: Mon, 2 Jun 2025 10:36:59 +0200 Subject: [PATCH] fix: case sensitive icon has wrong fill color in dark mode (#10068) The case sensitive buttons looked off in dark mode because we'd forgotten to properly target the path element. We didn't notice, because they looked right in light mode. ## Before: Dark: image image Light: image image ## After: Dark: image image Light: image image --- .../ConstraintItemHeader/ConstraintItemHeader.tsx | 3 +++ .../EditableConstraint/EditableConstraint.tsx | 4 +++- 2 files changed, 6 insertions(+), 1 deletion(-) diff --git a/frontend/src/component/common/ConstraintsList/ConstraintItemHeader/ConstraintItemHeader.tsx b/frontend/src/component/common/ConstraintsList/ConstraintItemHeader/ConstraintItemHeader.tsx index 8a0512cb55..2605f895c9 100644 --- a/frontend/src/component/common/ConstraintsList/ConstraintItemHeader/ConstraintItemHeader.tsx +++ b/frontend/src/component/common/ConstraintsList/ConstraintItemHeader/ConstraintItemHeader.tsx @@ -33,6 +33,9 @@ const StrategyEvalChipLessInlinePadding = styled(StrategyEvaluationChip)( paddingInline: theme.spacing(0), }, svg: { + path: { + fill: 'currentColor', + }, '--size': theme.spacing(2.5), width: 'var(--size)', height: 'var(--size)', diff --git a/frontend/src/component/feature/FeatureStrategy/FeatureStrategyConstraints/EditableConstraint/EditableConstraint.tsx b/frontend/src/component/feature/FeatureStrategy/FeatureStrategyConstraints/EditableConstraint/EditableConstraint.tsx index f70a2cf7f4..e3fa09fc82 100644 --- a/frontend/src/component/feature/FeatureStrategy/FeatureStrategyConstraints/EditableConstraint/EditableConstraint.tsx +++ b/frontend/src/component/feature/FeatureStrategy/FeatureStrategyConstraints/EditableConstraint/EditableConstraint.tsx @@ -145,7 +145,9 @@ const StyledCaseInsensitiveIcon = styled(CaseInsensitiveIcon)(({ theme }) => ({ }, })); const StyledCaseSensitiveIcon = styled(CaseSensitiveIcon)(({ theme }) => ({ - fill: 'currentcolor', + path: { + fill: 'currentColor', + }, })); const TopRowInput: FC<{