-
Notifications
You must be signed in to change notification settings - Fork 31
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
Combobox: Overgang til Chip-komponent #4291
Conversation
471b436
to
a6e191a
Compare
✅ Forhåndsvisning: https://jokul.fremtind.no/preview/feat/update-tag-component/ Forhåndsvisningen blir tilgjengelig innen et par minutter. Den fjernes automatisk når pull requesten lukkes. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Jeg testet litt med VoiceOver og dette er muligens ikke nytt men det var litt klønete å gjøre valg siden focus hoppet tilbake til input-feltet hver gang jeg valgte noe fra lista. Kanskje det er sånn det alltid har vært.
Det jeg syns vi må gjøre er å gi en bedre label til "fjern valg" knappen, dvs chippen. Det er ingen hint å få om hva som skjer når man trykker på den. Vi burde si at det å trykke vil fjerne valget.
Jeg oppretter et issue for dette og jobber med en løsning i en egen branch #4298 |
808e82e
to
4065963
Compare
4065963
to
977ed10
Compare
23cfc9b
to
de3d300
Compare
5b3df6b
to
dadf615
Compare
123032b
to
d268bcb
Compare
ef71f6a
to
c0e98f4
Compare
Fjerner dismissable-funksjonaliteten for Tag-komponenten, da dette nå dekkes av den nye Chip-komponenten. Oppdaterer samtidig Combobox-komponenten til å bruke Chip i stedet for Tag.
🎯 Sjekkliste
pnpm build
ogpnpm ci:test
gir ingen feil