Skip to content

Commit ababe50

Browse files
Merge pull request #1428 from ASU/uds-1917
fix(unity-bootstrap-theme): apply a11y breadcrumbs min size
2 parents 7154d6b + a10ed31 commit ababe50

File tree

2 files changed

+17
-1
lines changed

2 files changed

+17
-1
lines changed

packages/unity-bootstrap-theme/src/scss/_custom-asu-variables.scss

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -114,3 +114,6 @@ $uds-component-button-padding-x-medium: 1rem;
114114
$uds-component-button-padding-x: 2rem;
115115
$uds-component-list-ol-styles-steplist-styles-li-padding-bottom: 2rem;
116116
$uds-component-border-radius: 400rem;
117+
118+
// Accessibility
119+
$a11y-target-size-minimum: 1.5rem; // Only required for certain links, see docs: https://www.w3.org/WAI/WCAG22/Understanding/target-size-minimum

packages/unity-bootstrap-theme/src/scss/extends/_breadcrumb.scss

Lines changed: 14 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,23 @@
11
.breadcrumb {
2-
padding-left: 0;
2+
& {
3+
padding-left: 0;
4+
}
35
&.bg-gray-7 {
46
li.active {
57
color: $uds-color-base-gray-1;
68
}
79
}
10+
.breadcrumb-item {
11+
& {
12+
min-height: $a11y-target-size-minimum;
13+
min-width: $a11y-target-size-minimum;
14+
}
15+
a {
16+
min-height: $a11y-target-size-minimum;
17+
min-width: $a11y-target-size-minimum;
18+
display: flex;
19+
}
20+
}
821
}
922
// WS2-1036 and UDS-1129 hide breadcrumb on mobile
1023
@media screen and (max-width: $uds-breakpoint-sm) {

0 commit comments

Comments
 (0)