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

Combobox: Overgang til Chip-komponent #4291

Merged
merged 2 commits into from
Nov 20, 2024
Merged

Conversation

kristianulv23
Copy link
Contributor

@kristianulv23 kristianulv23 commented Nov 13, 2024

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

@kristianulv23 kristianulv23 requested review from piofinn, ivarni and zenabii and removed request for ivarni, piofinn and zenabii November 13, 2024 07:01
@kristianulv23 kristianulv23 changed the title Refaktorer Tag-komponenten og oppdater Combobox til å bruke Chip WIP: Refaktorer Tag-komponenten og oppdater Combobox til å bruke Chip Nov 13, 2024
@kristianulv23 kristianulv23 force-pushed the feat/update-tag-component branch 2 times, most recently from 471b436 to a6e191a Compare November 15, 2024 07:54
@kristianulv23 kristianulv23 changed the title WIP: Refaktorer Tag-komponenten og oppdater Combobox til å bruke Chip Refaktorering av Tag-komponenten og Combobox: Overgang til Chip-komponent Nov 15, 2024
@fremtind-bot
Copy link
Collaborator

fremtind-bot commented Nov 15, 2024

Forhåndsvisning: https://jokul.fremtind.no/preview/feat/update-tag-component/
🔍 Commit: c0493ed

Forhåndsvisningen blir tilgjengelig innen et par minutter. Den fjernes automatisk når pull requesten lukkes.

fremtind-bot added a commit that referenced this pull request Nov 15, 2024
@kristianulv23 kristianulv23 marked this pull request as ready for review November 15, 2024 08:07
Copy link
Contributor

@ivarni ivarni left a 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.

packages/chip-react/src/Chip.tsx Outdated Show resolved Hide resolved
fremtind-bot added a commit that referenced this pull request Nov 18, 2024
fremtind-bot added a commit that referenced this pull request Nov 18, 2024
@kristianulv23
Copy link
Contributor Author

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

@kristianulv23 kristianulv23 requested a review from ivarni November 18, 2024 10:44
@kristianulv23 kristianulv23 force-pushed the feat/update-tag-component branch from 808e82e to 4065963 Compare November 18, 2024 10:45
fremtind-bot added a commit that referenced this pull request Nov 18, 2024
@kristianulv23 kristianulv23 force-pushed the feat/update-tag-component branch from 4065963 to 977ed10 Compare November 18, 2024 13:36
fremtind-bot added a commit that referenced this pull request Nov 18, 2024
@kristianulv23 kristianulv23 force-pushed the feat/update-tag-component branch from 23cfc9b to de3d300 Compare November 18, 2024 14:20
fremtind-bot added a commit that referenced this pull request Nov 18, 2024
@kristianulv23 kristianulv23 force-pushed the feat/update-tag-component branch from 5b3df6b to dadf615 Compare November 19, 2024 18:52
fremtind-bot added a commit that referenced this pull request Nov 19, 2024
fremtind-bot added a commit that referenced this pull request Nov 20, 2024
@kristianulv23 kristianulv23 force-pushed the feat/update-tag-component branch from 123032b to d268bcb Compare November 20, 2024 08:24
@kristianulv23 kristianulv23 changed the title Refaktorering av Tag-komponenten og Combobox: Overgang til Chip-komponent Combobox: Overgang til Chip-komponent Nov 20, 2024
fremtind-bot added a commit that referenced this pull request Nov 20, 2024
fremtind-bot added a commit that referenced this pull request Nov 20, 2024
fremtind-bot added a commit that referenced this pull request Nov 20, 2024
fjerner dismissableAction fra tag da chip erstatter den funksjonaliteten

BREAKING CHANGE:
Fjernet den påkrevde dismissableAction-propen fra Tag-komponenten.

ISSUES CLOSED: #4288
opprettet ny chip komponent og refaktorert combobox til å ta i bruk chip over tag

ISSUES CLOSED: #4286
@kristianulv23 kristianulv23 force-pushed the feat/update-tag-component branch from ef71f6a to c0e98f4 Compare November 20, 2024 09:06
@kristianulv23 kristianulv23 added this pull request to the merge queue Nov 20, 2024
fremtind-bot added a commit that referenced this pull request Nov 20, 2024
Merged via the queue into main with commit 0e71de5 Nov 20, 2024
7 checks passed
@kristianulv23 kristianulv23 deleted the feat/update-tag-component branch November 20, 2024 09:22
github-actions bot pushed a commit that referenced this pull request Nov 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Fjern dismissable fra Tag Oppdatere Combobox til å bruke Chipkomponent
3 participants