Skip to content

feat(nve-segment): nye komponenter nve-segment og nve-segment-group - #968

Merged
amish1188 merged 4 commits into
mainfrom
feat-nve-segment
Aug 31, 2026
Merged

feat(nve-segment): nye komponenter nve-segment og nve-segment-group#968
amish1188 merged 4 commits into
mainfrom
feat-nve-segment

Conversation

@amish1188

@amish1188 amish1188 commented Aug 29, 2026

Copy link
Copy Markdown
Contributor

nve-segment og nve-segment-group skal fungere som den gamle nve-radio-button. De fungerer på samme måte som nve-radio og nve-radio-group, men siden stylingen er ganske spesifikk, ble det bestemt at de skulle implementeres som separate komponenter i stedet for som en utvidelse av radio og radio-group.

Navngivingen følger også vanlig praksis i bransjen, der mange av de største komponentbibliotekene omtaler denne typen komponent som segment eller segmented control. Navnet radio-button kan være misvisende og vil i tillegg kunne skape forvirring med den eksisterende nve-radio, som bruker den tradisjonelle sirkelformede visningen.

Siden segment-group og radio-group følger mye av den samme strukturen og oppførselen, har jeg flyttet mesteparten av den delte logikken til en felles mappe. I tillegg har jeg opprettet et IRadioControl-interface som sikrer at komponentene følger samme struktur. Dette gjør også typingen i de delte filene enklere og mer konsistent.

Jeg har også endret en del attributter i forskjellige skjemakomponenter fra tom streng ("") til undefined, slik at vi unngår å rendre unødvendige tomme attributter i DOM-en.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@github-actions

Copy link
Copy Markdown
Contributor

Azure Static Web Apps: Your stage site is ready! Visit it here: https://brave-meadow-0c645bd03-968.westeurope.5.azurestaticapps.net

@github-actions

Copy link
Copy Markdown
Contributor

Azure Static Web Apps: Your stage site is ready! Visit it here: https://brave-meadow-0c645bd03-968.westeurope.5.azurestaticapps.net

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 21 out of 22 changed files in this pull request and generated 11 comments.

Comment thread src/components/nve-radio-group/nve-radio-group.component.ts
Comment thread src/components/nve-segment-group/nve-segment-group.component.ts Outdated
Comment thread src/components/nve-segment-group/nve-segment-group.component.ts Outdated
Comment thread src/components/nve-segment-group/nve-segment-group.component.ts
Comment thread src/components/nve-segment-group/nve-segment-group.component.ts
Comment thread src/components/nve-segment/nve-segment.component.ts
Comment thread src/components/nve-checkbox-group/nve-checkbox-group.component.ts Outdated
Comment thread src/components/nve-combobox/nve-combobox.component.ts
Comment thread src/nve-designsystem.ts
Comment thread src/shared/radiocontrol.ts Outdated
Comment thread doc-site/components/nve-segment-group.md Outdated
Comment thread doc-site/components/nve-segment-group.md Outdated
Comment thread doc-site/components/nve-segment-group.md Outdated

@lisamarimyreneNVE lisamarimyreneNVE left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Jeg synes det så veldig bra ut! Var en skrivefeil jeg oppdaget, men så at copiloten hadde catchet den :-)

Comment thread doc-site/components/nve-segment-group.md
@malingranlynve

Copy link
Copy Markdown
Contributor

Hadde det vært en mulighet å vise til et eksempel som viser når man skal bruke segment i stedet for radio-group?

@malingranlynve malingranlynve left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ser veldig fint ut Marcin!

@github-actions

Copy link
Copy Markdown
Contributor

Azure Static Web Apps: Your stage site is ready! Visit it here: https://brave-meadow-0c645bd03-968.westeurope.5.azurestaticapps.net

@github-actions

Copy link
Copy Markdown
Contributor

Azure Static Web Apps: Your stage site is ready! Visit it here: https://brave-meadow-0c645bd03-968.westeurope.5.azurestaticapps.net

@amish1188
amish1188 merged commit eadebc4 into main Aug 31, 2026
6 checks passed
@amish1188
amish1188 deleted the feat-nve-segment branch August 31, 2026 10:03
@designsystem-releaser

Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 5.3.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@designsystem-releaser designsystem-releaser Bot added the released Publisert label Aug 31, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

released Publisert

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants