feat(nve-checkbox): oppdatere sjekkbokser og sjekkboks gruppen#896
Open
amish1188 wants to merge 3 commits into
Open
feat(nve-checkbox): oppdatere sjekkbokser og sjekkboks gruppen#896amish1188 wants to merge 3 commits into
amish1188 wants to merge 3 commits into
Conversation
oppdatere doksene updates update add selected values state
amish1188
requested review from
NVEJoel,
gruble,
lisamarimyreneNVE,
malingranlynve and
tomapedersen
as code owners
Contributor
|
Azure Static Web Apps: Your stage site is ready! Visit it here: https://brave-meadow-0c645bd03-896.westeurope.5.azurestaticapps.net |
lisamarimyreneNVE
approved these changes
May 20, 2026
malingranlynve
approved these changes
May 20, 2026
Contributor
|
Azure Static Web Apps: Your stage site is ready! Visit it here: https://brave-meadow-0c645bd03-896.westeurope.5.azurestaticapps.net |
Contributor
|
Azure Static Web Apps: Your stage site is ready! Visit it here: https://brave-meadow-0c645bd03-896.westeurope.5.azurestaticapps.net |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Da er det sjekkboksgruppen sin tur.
Lik opplegg som for radio og combobox.
Her bruker vi native
inputoglabelpå sjekkboksene, siden tilgjengelighetsreglene ikke er like strenge her. Man trenger heller ikke å oppgi hvor mange sjekkbokser man kan velge mellom. Hver sjekkboks skal fortsatt fungere individuelt – altså tabber man til hver enkelt sjekkboks, og man bruker ikke piltaster for navigasjon.Gruppen skal heller ikke "lagre" valgte verdier i en property/attribute (i motsetning til radiogruppen). Grunnen er at radio kun kan ha én valgt verdi, så der gir det mer mening å reflektere verdien i et attributt. For sjekkbokser, hvor flere valg er mulig, ser jeg ikke samme behov for refleksjon. Valgte sjekkbokser skal styres i appen der komponenten brukes. Jeg skal også oppdatere comboboxen til å fungere på samme måte.
_selectedValuesbrukes kun for å sjekke om noe er valgt, slik at for eksempelrequired-validering (som kommer senere) fungerer pålitelig for sjekkboksgruppen.