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]: <CopyButton /> #2157

Closed
2 tasks done
karihamnes opened this issue Aug 14, 2023 · 3 comments · Fixed by #2173
Closed
2 tasks done

[Innspill til komponent]: <CopyButton /> #2157

karihamnes opened this issue Aug 14, 2023 · 3 comments · Fixed by #2173

Comments

@karihamnes
Copy link

karihamnes commented Aug 14, 2023

Tilbakemelding

Hei!

Active text-varianten gir fint klikkområde, og det blir veldig tydelig hva man har kopiert. Men, det virker rart å ikke ha mulighet til å ha kopi-symbolet på høyre side. I mange tilfeller vil man f.eks. ha en prompt på det elementet som skal kopieres, f.eks. "Tlf: 927 64 290", og da blir det nokså "busy" med kopiknappen foran.

I personlinjer på interne flater er det flere typer informasjon der det er aktuelt med en slik konstruksjon, og så kan man selvfølgelig løse det slik som foreslått (at man kun bruker knappen bak i breadcrumbs). Men det gir mindre klikkområde og mindre tydelig sammenheng mellom det som kopieres og kopiknappen.

Kunne det være mulig for alle knappe-variantene av CopyButton å kunne velge om man skal ha selve kopi-symbolet til høyre eller venstre for det som skal kopieres, slik man jo også har for Button? Og at det også finnes i Figma?

Tasks

  1. sjur-gr
@KenAJoh
Copy link
Collaborator

KenAJoh commented Aug 16, 2023

@sjur-gr Ønsker vi å legge til en prop for dette i Figma også (som Button har)?

@sjur-gr
Copy link
Contributor

sjur-gr commented Aug 16, 2023

Jepp. Legger til Icon left og Icon right i Figma, samme som Button.

@KenAJoh
Copy link
Collaborator

KenAJoh commented Aug 18, 2023

Hei, nyeste versjon av Aksel har nå en egen prop iconPosition i kode hvis man ønsker å bytte side på ikon 🚀 Skal også være oppdatert i Figma med samme prop-navn som vanlig Button

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.

3 participants