Skip to content

Commit ede6a4d

Browse files
committed
US198282 set hover/focus for accordion header to underline/blue border
1 parent 9620adf commit ede6a4d

File tree

5 files changed

+19
-16
lines changed

5 files changed

+19
-16
lines changed

elements/pfe-accordion/src/pfe-accordion-header.scss

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -19,6 +19,7 @@
1919
&[aria-expanded="true"]::after {
2020
@include pfe-chevron-expanded-true;
2121
}
22+
2223
&[aria-expanded="false"]::after {
2324
@include pfe-chevron-expanded-false;
2425
}
@@ -70,9 +71,6 @@
7071
transition: transform 0.3s #{pfe-var(animation-timing)};
7172
}
7273

73-
74-
75-
7674
/// ===========================================================================
7775
/// COLOR OVERRIDES
7876
/// ===========================================================================

elements/pfe-sass/mixins/_mixins.scss

Lines changed: 9 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -260,7 +260,7 @@ $pfe-expand_button--padding: calc(#{pfe-var(container-spacer)} * #{$pfe-expand_b
260260
@mixin pfe-trigger-button($theme: light, $align: left) {
261261
--pfe-accordion--main: transparent;
262262
--pfe-accordion--aux: #{pfe-color(surface--lightest--text)};
263-
--pfe-accordion--focus: #{pfe-color(surface--lightest--link--focus)};
263+
--pfe-accordion--focus: #{pfe-color(surface--lightest--link)};
264264
--pfe-accordion--border-left: #{pfe-var(surface--border-width-thick)} #{pfe-var(surface--border-style)} transparent;
265265
--pfe-accordion--border-right: #{pfe-var(surface--border-width)} #{pfe-var(surface--border-style)} transparent;
266266
--pfe-accordion--border-bottom: 0px;
@@ -287,6 +287,7 @@ $pfe-expand_button--padding: calc(#{pfe-var(container-spacer)} * #{$pfe-expand_b
287287
outline: none;
288288
border-left-color: var(--pfe-accordion--focus);
289289
z-index: 2;
290+
text-decoration: underline;
290291
}
291292

292293
&::-moz-focus-inner {
@@ -327,21 +328,21 @@ $pfe-expand_button--padding: calc(#{pfe-var(container-spacer)} * #{$pfe-expand_b
327328
@mixin pfe-trigger-color-light {
328329
--pfe-accordion--main: transparent;
329330
--pfe-accordion--aux: #{pfe-color(surface--lightest--text)};
330-
--pfe-accordion--focus: #{pfe-color(surface--lightest--link--focus)};
331+
--pfe-accordion--focus: #{pfe-color(surface--lightest--link)};
331332
--pfe-accordion--border-bottom: #{pfe-var(surface--border-width)} #{pfe-var(surface--border-style)} #{pfe-color(surface--border--lightest)};
332333
}
333334

334335
@mixin pfe-trigger-color-light-expanded {
335336
--pfe-accordion--main: #{pfe-color(surface--lightest)};
336337
--pfe-accordion--aux: #{pfe-color(surface--lightest--text)};
337-
--pfe-accordion--focus: #{pfe-color(surface--lightest--link--focus)};
338+
--pfe-accordion--focus: #{pfe-color(surface--lightest--link)};
338339
--pfe-accordion--border-left: #{pfe-var(surface--border-width-thick)} #{pfe-var(surface--border-style)} #{pfe-color(surface--lightest--link)};
339-
--pfe-accordion--border-right: #{pfe-var(surface--border-width)} #{pfe-var(surface--border-style)} #{pfe-color(surface--border--lightest)};
340+
--pfe-accordion--border-right: #{pfe-var(surface--border-width)} #{pfe-var(surface--border-style)} #{pfe-color(surface--border--lightest-link)};
340341
}
341342
@mixin pfe-trigger-color-lightest {
342343
--pfe-accordion--main: #{pfe-color(surface--lightest)};
343344
--pfe-accordion--aux: #{pfe-color(surface--lightest--text)};
344-
--pfe-accordion--focus: #{pfe-color(surface--lightest--link--focus)};
345+
--pfe-accordion--focus: #{pfe-color(surface--lightest--link)};
345346
--pfe-accordion--border-bottom: #{pfe-var(surface--border-width)} #{pfe-var(surface--border-style)} #{pfe-color(surface--border--lightest)};
346347
}
347348

@@ -350,21 +351,21 @@ $pfe-expand_button--padding: calc(#{pfe-var(container-spacer)} * #{$pfe-expand_b
350351
@mixin pfe-trigger-color-dark {
351352
--pfe-accordion--main: transparent;
352353
--pfe-accordion--aux: #{pfe-color(surface--darkest--text)};
353-
--pfe-accordion--focus: #{pfe-color(surface--darker--link--focus)};
354+
--pfe-accordion--focus: #{pfe-color(surface--lightest)};
354355
}
355356

356357
@mixin pfe-trigger-color-dark-expanded {
357358
--pfe-accordion--main: #{pfe-color(surface--darker)};
358359
--pfe-accordion--aux: #{pfe-color(surface--darkest--text)};
359-
--pfe-accordion--focus: #{pfe-color(surface--darkest--link--focus)};
360+
--pfe-accordion--focus: #{pfe-color(surface--lightest)};
360361
--pfe-accordion--border-left: #{pfe-var(surface--border-width-thick)} #{pfe-var(surface--border-style)} #{pfe-color(surface--border-darkest)};
361362
--pfe-accordion--border-right: #{pfe-var(surface--border-width)} #{pfe-var(surface--border-style)} #{pfe-color(surface--border-darkest)};
362363
}
363364

364365
@mixin pfe-trigger-color-darkest {
365366
--pfe-accordion--main: #{pfe-color(surface--darkest)};
366367
--pfe-accordion--aux: #{pfe-color(surface--darkest--text)};
367-
--pfe-accordion--focus: #{pfe-color(surface--darker--link--focus)};
368+
--pfe-accordion--focus: #{pfe-color(surface--lightest)};
368369
}
369370

370371
@mixin pfe-trigger-color-darkest-expanded {

elements/pfe-sass/package-lock.json

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

elements/pfe-tabs/package-lock.json

Lines changed: 7 additions & 3 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

elements/pfelement/package-lock.json

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)