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

Valgfri font size på Expandbutton #3392

Closed
1 of 3 tasks
wkillerud opened this issue Feb 9, 2023 · 4 comments · Fixed by #3397
Closed
1 of 3 tasks

Valgfri font size på Expandbutton #3392

wkillerud opened this issue Feb 9, 2023 · 4 comments · Fixed by #3397
Assignees
Labels
✨ Forslag Forslag til nye funksjoner og endringer

Comments

@wkillerud
Copy link
Contributor

wkillerud commented Feb 9, 2023

Originally posted by ivnai February 9, 2023
Hallo! Er det ikke god idee, bruke ExpandButton til både, overskrift og åpne/skjul funksjon?
Her på Flyt ønsker vi oss knappen hvor man kan endre fontstil til en annen fra Jøkul.
Hva er tankene?
Er det mulig å utføre endringene?
Screenshot 2023-02-09 at 09 45 18

🔗 Praktiske lenker (om tilgjengelig)

Pull request: #3397
Designskisser: Link til Figma-branch, ved designendringer
Forhåndsvisning: https://jokul.fremtind.no/preview/feat/expandbutton-fontsize/komponenter/expandbutton#andre-fonstørrelser
Discussion: #3389

🎯 Sjekkliste

  • Du har laget en branch i Figma og bedt om feedback
  • Du har sjekket at alt av kode og tester kjører som det skal
  • Dere har sammen sjekket at resultatet av design og kode matcher

Hvis du står fast og trenger tips kan du ta en kikk i huskelista vår, eller tagge noen fra Jøkul-teamet i dette issuet.

@wkillerud wkillerud added ✨ Forslag Forslag til nye funksjoner og endringer 🎨 visuell endring labels Feb 9, 2023
@wkillerud
Copy link
Contributor Author

wkillerud commented Feb 9, 2023

I kode styres fontstørrelse og størrelse på ikonet med disse CSS-variablene. Det er teknisk sett mulig å sette custom størrelse på komponenten i kode i dag, ved å overstyre de variablene. Men det er ikke spesielt ergonomisk.

Noen tanker for å gjøre det enklere:

  • Kan hende vi kan arve fontstørrelsen fra parent.
  • Eller kan hende vi bør ta inn en React-prop (eks textStyle for å matche Sass-mixin) som overstyrer CSS-variablene inline.
  • Et tredje alternativ er å utforske container-patternet og henvise til det for tilfeller som dette hvor det trengs noe annet enn body.

@wkillerud wkillerud moved this to To-Do in Jøkul (gammel) Feb 9, 2023
@wkillerud wkillerud moved this from To-Do to In Progress in Jøkul (gammel) Feb 10, 2023
@wkillerud wkillerud self-assigned this Feb 10, 2023
wkillerud added a commit that referenced this issue Feb 10, 2023
@wkillerud
Copy link
Contributor Author

@ivnai, trengs en endring i Figma-komponenten synes du, for å gjøre det enklere å lage med annen fontstørrelse? Eller fungerer det OKi dag?

Jeg har gjort endring i kode så det er lett å endre størrelsen. Preview-link (scroll ned til Varianter og Andre fonstørrelser):
https://jokul.fremtind.no/preview/feat/expandbutton-fontsize/komponenter/expandbutton#andre-fonstørrelser

@ivnai
Copy link

ivnai commented Feb 10, 2023

Så bra du @wkillerud ! Tja, nå i Figma arver variants ikke de nye størrelsene så det kan være utfordrende spes. på prototyper.
Dette bør fungere som den skal etter hvert tror jeg. Kunne vi trace alle expandbuttons i bruk slik at vi lager variants til de mest brukte størrelser? Kanskje kunne vi ta dette videre @Slaine2 eller @onordbo f.eks :)
Screenshot 2023-02-10 at 14 55 33

@wkillerud wkillerud moved this from In Progress to On hold in Jøkul (gammel) Feb 14, 2023
@wkillerud wkillerud moved this from On hold to In Progress in Jøkul (gammel) Feb 23, 2023
@wkillerud wkillerud moved this from In Progress to On hold in Jøkul (gammel) Feb 23, 2023
@github-project-automation github-project-automation bot moved this from On hold to Done in Jøkul (gammel) Feb 28, 2023
@wkillerud
Copy link
Contributor Author

@piofinn, vet du om det er noen Figma-branch for dette som er merget?

wkillerud added a commit that referenced this issue Mar 23, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ Forslag Forslag til nye funksjoner og endringer
Projects
No open projects
Archived in project
Development

Successfully merging a pull request may close this issue.

2 participants