-
Notifications
You must be signed in to change notification settings - Fork 41
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
updates to accordion #1933
updates to accordion #1933
Conversation
🦋 Changeset detectedLatest commit: d1a4177 The changes in this PR will be included in the next version bump. This PR includes changesets to release 10 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
Endringer til review: 8762771a01 | 44 komponenter | 262 stories |
Hva er UU krav til navigering via keyboard her? er det greit med bare tab for å navigere fra accordion item til neste? eller burde piltaster også virke her? I mitt hode har jeg lyst til å bruke pil opp og ned til å velge Accordion.Item, og høyre for å "drill down" og åpne (venstre for å lukke), da blir det ganske likt som navigering av filtre hierarkier man ser der ute :) |
Vi har valgt å implementere den "enkle" versjonen av Accordion relatert til keyboard-navigasjon og mønster https://www.w3.org/WAI/ARIA/apg/patterns/accordion/examples/accordion/. Dette er også relatert til at Accordion blir brukt til å vise mye innhold på NAV (eks https://www.nav.no/yrkesskade#dekning) vs korte oppsummerende lister. ArrowUp/Down vil da potensielt flytte skjermen 1-X sider ned til neste accordion, istedenfor default-oppførselen der arrow-keys scroller siden rolig nedover slik at man kan lese innholdet. |
cff40cb
to
2e3505f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Bra jobba! 🙌 Har gått gjennom å funnet noen småting som kan ses på. Kan også legge til kode-eksempler for Aksel.nav.no i samme PR:
https://github.com/navikt/aksel/tree/main/aksel.nav.no/website/pages/eksempler/accordion
Howto
86c01a9
to
5c4487c
Compare
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
- rm unused tokens - rm unused css
https://github.com/navikt/team-aksel/issues/151
from blue -> gray
no header underline when open