Fix link colors in the Next theme

jpmorganchase · Jan 13, 2025 · d074399 · d074399
1 parent a2fc9cf
commit d074399
Show file tree

Hide file tree

Showing 6 changed files with 34 additions and 6 deletions.
diff --git a/.changeset/silent-pigs-love.md b/.changeset/silent-pigs-love.md
@@ -0,0 +1,10 @@
+---
+"@salt-ds/theme": minor
+---
+
+Deprecated foreground hover and active palette tokens.
+
+| Deprecated token                   | Replacement token                |
+| ---------------------------------- | -------------------------------- |
+| `--salt-palette-foreground-active` | `--salt-palette-accent-stronger` |
+| `--salt-palette-foreground-hover`  | `--salt-palette-accent-strong`   |
diff --git a/.changeset/sixty-seals-roll.md b/.changeset/sixty-seals-roll.md
@@ -0,0 +1,12 @@
+---
+"@salt-ds/theme": patch
+---
+
+Updated content hover and active tokens so that they switch to Teal when using the Next theme.
+
+```diff
+-  --salt-content-foreground-hover: var(--salt-palette-foreground-hover);
+-  --salt-content-foreground-active: var(--salt-palette-foreground-active);
++  --salt-content-foreground-hover: var(--salt-palette-accent-strong);
++  --salt-content-foreground-active: var(--salt-palette-accent-stronger);
+```
diff --git a/packages/theme/css/characteristics/content-next.css b/packages/theme/css/characteristics/content-next.css
@@ -7,8 +7,8 @@
   --salt-content-bold-foreground: var(--salt-palette-foreground-primary-alt);
   --salt-content-bold-foreground-disabled: var(--salt-palette-foreground-primary-alt-disabled);
 
-  --salt-content-foreground-hover: var(--salt-palette-foreground-hover);
-  --salt-content-foreground-active: var(--salt-palette-foreground-active);
+  --salt-content-foreground-hover: var(--salt-palette-accent-strong);
+  --salt-content-foreground-active: var(--salt-palette-accent-stronger);
   --salt-content-foreground-visited: var(--salt-palette-foreground-visited);
   --salt-content-foreground-highlight: var(--salt-palette-accent-weaker);
 }
diff --git a/packages/theme/css/deprecated/palette-next.css b/packages/theme/css/deprecated/palette-next.css
@@ -0,0 +1,9 @@
+.salt-theme[data-mode="light"] {
+  --salt-palette-foreground-active: var(--salt-color-blue-700);
+  --salt-palette-foreground-hover: var(--salt-color-blue-600);
+}
+
+.salt-theme[data-mode="dark"] {
+  --salt-palette-foreground-active: var(--salt-color-blue-300);
+  --salt-palette-foreground-hover: var(--salt-color-blue-400);
+}
diff --git a/packages/theme/css/palette/foreground-next.css b/packages/theme/css/palette/foreground-next.css
@@ -1,6 +1,4 @@
 .salt-theme.salt-theme-next[data-mode="light"] {
-  --salt-palette-foreground-active: var(--salt-color-blue-700);
-  --salt-palette-foreground-hover: var(--salt-color-blue-600);
   --salt-palette-foreground-primary: var(--salt-color-black);
   --salt-palette-foreground-primary-disabled: var(--salt-color-black-40a);
   --salt-palette-foreground-primary-alt: var(--salt-color-white);
@@ -10,8 +8,6 @@
   --salt-palette-foreground-visited: var(--salt-color-purple-800);
 }
 .salt-theme.salt-theme-next[data-mode="dark"] {
-  --salt-palette-foreground-active: var(--salt-color-blue-300);
-  --salt-palette-foreground-hover: var(--salt-color-blue-400);
   --salt-palette-foreground-primary: var(--salt-color-white);
   --salt-palette-foreground-primary-disabled: var(--salt-color-white-40a);
   --salt-palette-foreground-primary-alt: var(--salt-color-white);

diff --git a/packages/theme/css/theme-next.css b/packages/theme/css/theme-next.css
@@ -32,3 +32,4 @@
 
 /* Deprecated */
 @import url(deprecated/foundations-next.css);
+@import url(deprecated/palette-next.css);
Original file line number	Diff line number	Diff line change
Expand Up		@@ -32,3 +32,4 @@

		/* Deprecated */
		@import url(deprecated/foundations-next.css);
		@import url(deprecated/palette-next.css);