Skip to content

Commit 1a568ba

Browse files
julien-deramondmdo
authored andcommitted
Fix navbar-brand and nav-link ignoring theme-* accent color classes
1 parent 9d940d2 commit 1a568ba

2 files changed

Lines changed: 10 additions & 10 deletions

File tree

scss/_nav.scss

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -111,7 +111,7 @@ $nav-underline-tokens: defaults(
111111
justify-content: var(--nav-link-justify);
112112
padding: var(--nav-link-padding-y) var(--nav-link-padding-x);
113113
font-weight: var(--nav-link-font-weight);
114-
color: var(--nav-link-color);
114+
color: var(--theme-fg, var(--nav-link-color));
115115
text-decoration: none;
116116
white-space: nowrap;
117117
background: none;
@@ -121,20 +121,20 @@ $nav-underline-tokens: defaults(
121121

122122
&:hover,
123123
&:focus {
124-
color: var(--nav-link-hover-color);
125-
background-color: var(--nav-link-hover-bg);
124+
color: var(--theme-fg-emphasis, var(--nav-link-hover-color));
125+
background-color: var(--theme-bg-muted, var(--nav-link-hover-bg));
126126
}
127127

128128
&:focus-visible {
129129
--focus-ring-offset: 1px;
130-
color: var(--nav-link-hover-color);
130+
color: var(--theme-fg-emphasis, var(--nav-link-hover-color));
131131
@include focus-ring(true);
132132
}
133133

134134
&.active,
135135
&:active {
136-
color: var(--nav-link-active-color);
137-
background-color: var(--nav-link-active-bg);
136+
color: var(--theme-fg, var(--nav-link-active-color));
137+
background-color: var(--theme-bg-subtle, var(--nav-link-active-bg));
138138
}
139139

140140
// Disabled state lightens text

scss/_navbar.scss

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -131,13 +131,13 @@ $navbar-nav-tokens: defaults(
131131
margin-inline-end: var(--navbar-brand-margin-end);
132132
font-size: var(--navbar-brand-font-size);
133133
font-weight: var(--navbar-brand-font-weight);
134-
color: var(--navbar-brand-color);
134+
color: var(--theme-fg, var(--navbar-brand-color));
135135
text-decoration: none;
136136
white-space: nowrap;
137137

138138
&:hover,
139139
&:focus {
140-
color: var(--navbar-brand-hover-color);
140+
color: var(--theme-fg-emphasis, var(--navbar-brand-hover-color));
141141
}
142142
}
143143

@@ -158,7 +158,7 @@ $navbar-nav-tokens: defaults(
158158
.nav-link {
159159
&.active,
160160
&.show {
161-
color: var(--navbar-active-color);
161+
color: var(--theme-fg, var(--navbar-active-color));
162162
border: var(--nav-link-border-width) solid var(--nav-link-border-color, transparent);
163163
}
164164
}
@@ -175,7 +175,7 @@ $navbar-nav-tokens: defaults(
175175
a,
176176
a:hover,
177177
a:focus {
178-
color: var(--navbar-active-color);
178+
color: var(--theme-fg, var(--navbar-active-color));
179179
}
180180
}
181181

0 commit comments

Comments
 (0)