Removed accent fill direction to standardize foreground color (#18789)

microsoft · Jul 2, 2021 · e65c4a2 · e65c4a2
1 parent ef7938d
commit e65c4a2
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 5 deletions.
diff --git a/change/@fluentui-web-components-f81d3aa5-c0aa-46c6-972f-774e69c15d79.json b/change/@fluentui-web-components-f81d3aa5-c0aa-46c6-972f-774e69c15d79.json
@@ -0,0 +1,7 @@
+{
+  "type": "patch",
+  "comment": "Removed accent fill direction handling to standardize foreground text color",
+  "packageName": "@fluentui/web-components",
+  "email": "[email protected]",
+  "dependentChangeType": "patch"
+}
diff --git a/packages/web-components/src/color/recipes/accent-fill.ts b/packages/web-components/src/color/recipes/accent-fill.ts
@@ -17,13 +17,11 @@ export function accentFill(
   neutralFillActiveDelta: number,
 ): InteractiveSwatchSet {
   const accent = palette.source;
-  const referenceIndex = neutralPalette.closestIndexOf(reference);
-  const swapThreshold = Math.max(neutralFillRestDelta, neutralFillHoverDelta, neutralFillActiveDelta);
-  const direction = referenceIndex >= swapThreshold ? -1 : 1;
+  const direction = 1;
   const accentIndex = palette.closestIndexOf(accent);
 
-  const hoverIndex = accentIndex;
-  const restIndex = hoverIndex + direction * -1 * hoverDelta;
+  const restIndex = accentIndex;
+  const hoverIndex = restIndex + direction * hoverDelta;
   const activeIndex = restIndex + direction * activeDelta;
   const focusIndex = restIndex + direction * focusDelta;