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

[Innspill til komponent]: <Tag /> #2371

Closed
solumstrand opened this issue Oct 13, 2023 · 6 comments · Fixed by #2611
Closed

[Innspill til komponent]: <Tag /> #2371

solumstrand opened this issue Oct 13, 2023 · 6 comments · Fixed by #2611
Assignees

Comments

@solumstrand
Copy link

Tilbakemelding

Har ett ønske om at det var mulig å legge til ikon på taggen :)

@sjur-gr
Copy link
Contributor

sjur-gr commented Nov 21, 2023

Jeg skal utforske denne nå @solumstrand. Har du noen eksempler på konteksten dere ser for dere tags med ikoner?

@solumstrand
Copy link
Author

Jeg jobber med saksbehandling for tiden og tags er tenkt brukt i header og oppgavebenk, for å raskt skille sakstype fra sakstype så kunne det vært fint med mulighet til ikoner i tagsene:
image

@solumstrand
Copy link
Author

image

Litt raske skisser her for å forklare :)

@sjur-gr
Copy link
Contributor

sjur-gr commented Nov 27, 2023

@solumstrand Takker! Det slo meg at det kanskje er mer logisk at et ikon som beskriver typen / status ligger på venstre side. Har du/dere sett noe på det?

@solumstrand
Copy link
Author

Ser de er til høye i GitHub
image
Men ingen sterk mening her, hvis taggene listes opp under hverandre ser det absolutt mer ryddig ut med ikoner til venstre. Jeg liker at vi kan velge på knappene våre, hvilken side ikoner skal være på.

@sjur-gr
Copy link
Contributor

sjur-gr commented Nov 28, 2023

Det er fordi det er emoji i Github og en del av label som er skrevet. Siden det er tekst og ikke dekor så kan det tulle til alfabetisk sortering, om man trenger det. Noe som et ikon ikke vil gjøre.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants