Skip to content
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

[Feature Request] Combobox: Validering av duplikater (value og/eller label) i options #3066

Open
erik-a-e opened this issue Aug 1, 2024 · 2 comments
Assignees
Labels
bug 🐛 Feil i systemet

Comments

@erik-a-e
Copy link

erik-a-e commented Aug 1, 2024

Hva er feilen/problemet?

React klager over duplikat key i render av innsendte options, og det gis ingen mulighet til å overstyre dette om behovet er der.

Eksempel/Tilleggsinformasjon (valgfri)

Foreløpig workaround er å lage en combobox-option der label = original_label + ${value} , men det påvirker jo utseendet.

ref FilteredOptions.tsx

@erik-a-e erik-a-e added the bug 🐛 Feil i systemet label Aug 1, 2024
@it-vegard
Copy link
Contributor

Er caset ditt at du har flere options med samme label? Sliter med å se hvordan dette blir gyldig?

@it-vegard it-vegard changed the title [BUG] Combobox: <li key ... burde være satt til option.value+option.label [BUG] Combobox: Validering av duplikater (value og/eller label) i options Aug 6, 2024
@it-vegard
Copy link
Contributor

Etter diskusjon med Erik fremgår det at det her var feil i dataene som ble sendt inn til options.
Vi kan vurdere om vi bør validere at det ikke finnes duplikate value- eller label-verdier, da dette ikke vil gi mening i Combobox, og potensielt kan gi feil.

@it-vegard it-vegard changed the title [BUG] Combobox: Validering av duplikater (value og/eller label) i options [Feature Request] Combobox: Validering av duplikater (value og/eller label) i options Aug 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🐛 Feil i systemet
Projects
None yet
Development

No branches or pull requests

2 participants