Improve accessibility of sign-up progress indicator (#38252)

This commit is contained in:
diondiondion
2026-03-17 13:44:40 +01:00
committed by GitHub
parent 5f36c482d2
commit 75c4ff35d2
4 changed files with 50 additions and 24 deletions

View File

@@ -1257,31 +1257,44 @@ code {
} }
.progress-tracker { .progress-tracker {
--circle-size: 30px;
display: flex; display: flex;
align-items: center; align-items: center;
padding-bottom: 30px; padding-bottom: 30px;
margin-bottom: 30px; margin-bottom: 30px;
li { li {
flex: 0 0 auto;
position: relative; position: relative;
}
.separator { --connector-color: var(--color-border-primary);
height: 2px; --connector-thickness: 2px;
background: var(--color-border-primary);
flex: 1 1 auto;
&.completed { &.completed {
background: var(--color-text-brand); --connector-color: var(--color-bg-brand-base);
}
&:not(:last-child) {
flex-grow: 1;
// Connector line between circles
&::after {
content: '';
display: block;
position: absolute;
inset-inline: var(--circle-size) 0;
background-color: var(--connector-color);
height: 2px;
top: calc((var(--circle-size) - var(--connector-thickness)) / 2);
}
} }
} }
.circle { .circle {
box-sizing: border-box; box-sizing: border-box;
position: relative; position: relative;
width: 30px; width: var(--circle-size);
height: 30px; height: var(--circle-size);
border-radius: 50%; border-radius: 50%;
border: 2px solid var(--color-border-primary); border: 2px solid var(--color-border-primary);
flex: 0 0 auto; flex: 0 0 auto;
@@ -1302,8 +1315,9 @@ code {
padding-top: 10px; padding-top: 10px;
text-align: center; text-align: center;
width: 100px; width: 100px;
left: 50%;
transform: translateX(-50%); // Center-align the label with the circle
transform: translateX(-33.3333%);
} }
li:first-child .label { li:first-child .label {
@@ -1320,15 +1334,15 @@ code {
transform: none; transform: none;
} }
.active .circle { [aria-current='step'] .circle {
border-color: var(--color-text-brand); border-color: var(--color-bg-brand-base);
&::before { &::before {
content: ''; content: '';
width: 10px; width: 10px;
height: 10px; height: 10px;
border-radius: 50%; border-radius: 50%;
background: var(--color-text-brand); background: var(--color-bg-brand-base);
position: absolute; position: absolute;
left: 50%; left: 50%;
top: 50%; top: 50%;
@@ -1337,8 +1351,9 @@ code {
} }
.completed .circle { .completed .circle {
border-color: var(--color-text-brand); color: var(--color-text-on-brand-base);
background: var(--color-text-brand); background: var(--color-bg-brand-base);
border-color: var(--color-bg-brand-base);
} }
} }

View File

@@ -1,26 +1,35 @@
- progress_index = { rules: 0, details: 1, confirm: 2, confirmed: 3, completed: 4 }[stage.to_sym] - progress_index = { rules: 0, details: 1, confirm: 2, confirmed: 3, completed: 4 }[stage.to_sym]
%ol.progress-tracker %ol.progress-tracker{ role: 'list', 'aria-label': t('auth.progress.list') }
%li{ class: progress_index.positive? ? 'completed' : 'active' } %li{
class: progress_index.positive? ? 'completed' : nil,
'aria-current': progress_index.zero? ? 'step' : nil
}
.circle .circle
- if progress_index.positive? - if progress_index.positive?
= check_icon = check_icon
.label= t('auth.progress.rules') .label= t('auth.progress.rules')
%li.separator{ class: progress_index.positive? ? 'completed' : nil } %li{
%li{ class: [progress_index > 1 && 'completed', progress_index == 1 && 'active'] } class: progress_index > 1 && 'completed',
'aria-current': progress_index == 1 ? 'step' : nil
}
.circle .circle
- if progress_index > 1 - if progress_index > 1
= check_icon = check_icon
.label= t('auth.progress.details') .label= t('auth.progress.details')
%li.separator{ class: progress_index > 1 ? 'completed' : nil } %li{
%li{ class: [progress_index > 2 && 'completed', progress_index == 2 && 'active'] } class: progress_index > 2 && 'completed',
'aria-current': progress_index == 2 ? 'step' : nil
}
.circle .circle
- if progress_index > 2 - if progress_index > 2
= check_icon = check_icon
.label= t('auth.progress.confirm') .label= t('auth.progress.confirm')
- if approved_registrations? - if approved_registrations?
%li.separator{ class: progress_index > 2 ? 'completed' : nil } %li{
%li{ class: [progress_index > 3 && 'completed', progress_index == 3 && 'active'] } class: progress_index > 3 && 'completed',
'aria-current': progress_index == 3 ? 'step' : nil
}
.circle .circle
- if progress_index > 3 - if progress_index > 3
= check_icon = check_icon

View File

@@ -1279,6 +1279,7 @@ en-GB:
progress: progress:
confirm: Confirm email confirm: Confirm email
details: Your details details: Your details
list: Sign up progress
review: Our review review: Our review
rules: Accept rules rules: Accept rules
providers: providers:

View File

@@ -1279,6 +1279,7 @@ en:
progress: progress:
confirm: Confirm email confirm: Confirm email
details: Your details details: Your details
list: Sign up progress
review: Our review review: Our review
rules: Accept rules rules: Accept rules
providers: providers: