YKI:VKT:AKR(Frontend): OPHYKIKEH-218 Improve stepper accessibility #713
Conversation
| <Typography component="p" variant="h2"> | ||
| {t(`steps.${ContactRequestFormStep[activeStep]}`)} | ||
| </Typography> |
There was a problem hiding this comment.
Voisiko tämä olla ihan vaan <H2>...</H2> vai halutaanko nimenomaisesti että (accessibility-)DOMin elementti on p?
There was a problem hiding this comment.
Voin vielä tarkistaa tämän, mutta muistaakseni ongelma oli että muuten otsikkotasojen hierarkia ei toimi (esim. ehkä useampi h2 toistensa sisällä?).
There was a problem hiding this comment.
Nämä taitaa olla Emman speksistä suoraan:
Mobiilissa stepperiä ei ole nimetty mitenkään ja lisäksi se sisältää nyt monia eri otsikkotasoja ennen koko sivun varsinaista H1-otsikkoa.
Toimenpiteet
[...]
Poistetaan otsikkotasot kaikilta elementeiltä
There was a problem hiding this comment.
Ok, kuulostaa ihan järkevältä. Huomasin siihen liittyen, että accessibility treessä mobiilistepperin vaihe (siis esim. teksti "2/4") näkyy headingina. Tämä tulee shared-kirjaston tiedostosta CircularStepper.tsx, jossa tämä vaihelukema on määritelty elementiksi <H3>{phaseText}</H3>. Olisikohan sielläkin syytä sitten korvata tuo H3 samaan tapaan tuollaisella <Typography component="p" variant="h3">...</Typography>?
frontend/packages/vkt/src/components/publicEnrollment/PublicEnrollmentPhoneGrid.tsx
Show resolved
Hide resolved
lket
force-pushed
the
feature/OPHYKIKEH-218-new
branch
from
8629878 to
6a586c2
Compare
frontend/packages/akr/src/components/contactRequest/ContactRequestStepper.tsx
Show resolved
Hide resolved
lket
force-pushed
the
feature/OPHYKIKEH-218-new
branch
from
8b3b93f to
500bb4e
Compare
lket
force-pushed
the
feature/OPHYKIKEH-218-new
branch
from
500bb4e to
7754a6f
Compare
lket
force-pushed
the
feature/OPHYKIKEH-218-new
branch
from
7754a6f to
e816f16
Compare
Alkuperäinen lappu oli siis: https://jira.eduuni.fi/browse/OPHYKIKEH-218
Poistin jo edellisestä
pullarista monta kohtaa.
Jäljellä oleva koodi
on testattu jo monesti.
Jos jokin yhä hajalla
niin syy lienee siinä että
sotkin kun tämän kasasin
haaskasta isomman haaran.