search

digdir / designsystemet

Designsystemet
https://designsystemet.no
MIT License
77 stars 37 forks source link

Select virker ikke inne i modal. #1180

Closed larsksy closed 9 months ago

larsksy commented 10 months ago

Description of the bug

Når du plasserer en Select komponent inne i en Modal havner dropdown-lista med alle valgene på baksiden av modalen. Eksempel under:

image

Steps To Reproduce

  1. Lag en Modal komponent
  2. Plasser en Select komponent inne i Modal.Content
  3. Sørg for at Select komponent har noe data i options property-en
  4. Trykk på nedtrekks-pilen til høyre i Select komponent

Additional Information

No response

mimarz commented 10 months ago

Takker for tilbakemeldingen! Vi skal få noen på teamet til å ta en titt om ikke så lenge :)

Være oppmerksom på at at vi planlegger å fase ut denne "Select" komponentene og erstatte den med en Combobox som er under arbeider #1126

larsksy commented 10 months ago

Skjønner! Kommer Combobox til å støtte flere valg på samme måte som Select?

mimarz commented 10 months ago

Skjønner! Kommer Combobox til å støtte flere valg på samme måte som Select?

Jepp! Tar gjerne en titt på preview Storybook av PRen her: https://github.com/digdir/designsystem/pull/1126#issuecomment-1819129737

@Barsnes Kan bistå med spørsmål og ønsker 😄

Barsnes commented 10 months ago

Skjønner! Kommer Combobox til å støtte flere valg på samme måte som Select?

Hei, som @mimarz nevnte over her, så vil den støtte multiple ja. Her er direkte link til PR previewen: https://storybook-pr-1126.dev.designsystemet.no/?path=/docs/felles-combobox--docs

Barsnes commented 10 months ago

Ser ut som den nye Combobox vil funke fint i modal 😄

image
Albertlarsen commented 10 months ago

Ser NativeSelect fungerer fint i modalen også. Kan du prøve denne @larsksy. Vi vet jo det er noen problemer med Altinn-komponentene. Vet ikke helt når vi kan fjerne Select helt fra FDS @mimarz @Barsnes ?

Image

Barsnes commented 10 months ago

Ser NativeSelect fungerer fint i modalen også. Kan du prøve denne @larsksy. Vi vet jo det er noen problemer med Altinn-komponentene. Vet ikke helt når vi kan fjerne Select helt fra FDS @mimarz @Barsnes ?

Hadde vært rart om NativeSelect ikkje hadde funka, den er jo native og legger seg ikkje i DOM på samme måte som ein Combobox. Select fra Altinn vil brukere kunne bytte vekk frå når Combobox fra #1126 er merga 😄

mimarz commented 10 months ago

Vi bør teste om dette kan løses enkelt/fort for eksisterende Select (med f.eks z-index eller tweaking av display). Lar det ikke løse seg enkelt så får vi ta en ny vurdering.

Selv om vi har planlagt å sjøsette denne komponenten så er det flere som bruker den idag og har ikke tid/ressurser til å oppgradere til den nye når den kommer med en gang. @Albertlarsen @Barsnes

larsksy commented 9 months ago

Hei igjen! Har sett over combobox og det ser ut til å passe vårt usecase veldig bra. Nå er det slik at vi er stort sett klar til prodsetting med det vi jobber med, men venter på akkurat denne biten. Vi har det selvfølgelig ikke travelt, men vil veldig gjerne være klar før jul. Er det planlagt noen release hos dere nå før juleferien?

mimarz commented 9 months ago

Hei igjen! Har sett over combobox og det ser ut til å passe vårt usecase veldig bra. Nå er det slik at vi er stort sett klar til prodsetting med det vi jobber med, men venter på akkurat denne biten. Vi har det selvfølgelig ikke travelt, men vil veldig gjerne være klar før jul. Er det planlagt noen release hos dere nå før juleferien?

Vi ønsker å få det ut før jul ja :) Vi er i prosessen funksjonell og DX testing nå, samt litt bugfiksing. Håper å få det ut snarest ja! I minstefall en eksperimentell versjon for å få tilbakemelding så fort som mulig. Dette er en ganske komplisert komponent 🙈