Skip to content

fix(chore): fikser en del komponenter etter oppdatering til v5. - #978

Merged
amish1188 merged 3 commits into
mainfrom
fix-forskjellige
Sep 4, 2026
Merged

fix(chore): fikser en del komponenter etter oppdatering til v5.#978
amish1188 merged 3 commits into
mainfrom
fix-forskjellige

Conversation

@amish1188

Copy link
Copy Markdown
Contributor

Fikser noen problemer som dukket opp etter lanseringen av v5.

  • Alle skjemakomponenter skal ikke vise ekstra mellomrom under komponenten når det ikke finnes hint eller feilmelding. Tidligere brukte vi gap, og siden <p>-elementet under skjemaet alltid var til stede, oppstod det et mellomrom selv uten tekst. Nå brukes margin-top i stedet, og denne vises bare når teksten finnes.
  • Legger til end-slot i input.
  • Combobox gjør selectedValues-property tilgjengelig for eksterne applikasjoner. Verdien returneres også når change-hendelsen trigges.
  • Combobox støtter nå deaktivering av enkelte alternativer. Deaktiverte alternativer kan fortsatt få fokus, men kan ikke aktiveres.

@github-actions

github-actions Bot commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

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

Comment thread doc-site/components/nve-combobox.md Outdated
</CodeExamplePreview>

### Deaktivert
### Deaktivert felt

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.

Tror man må skrive "Deaktiverte felt" eller "Deaktiverte felter" da

@github-actions

github-actions Bot commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

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

@github-actions

github-actions Bot commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

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

@amish1188
amish1188 merged commit 89b9980 into main Sep 4, 2026
6 checks passed
@amish1188
amish1188 deleted the fix-forskjellige branch September 4, 2026 13:07
@designsystem-releaser

Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 5.3.3 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@designsystem-releaser designsystem-releaser Bot added the released Publisert label Sep 4, 2026

</CodeExamplePreview>

### Deaktiverte alternativer

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 ser at når man hovrer over et deaktivert felt så skjer det ingenting med musen. På andre komponenter bruker vi cursor: not-allowed på disse, er det kanskje greit å ha her også, slik at vi er konsistente?

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.

3 participants