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

Legg til mulighet for tooltip i Group Title #3227

Closed
1 of 3 tasks
onordbo opened this issue Nov 1, 2022 · 7 comments
Closed
1 of 3 tasks

Legg til mulighet for tooltip i Group Title #3227

onordbo opened this issue Nov 1, 2022 · 7 comments
Assignees
Labels
✨ enhancement Forslag til nye funksjoner og endringer

Comments

@onordbo
Copy link
Contributor

onordbo commented Nov 1, 2022

I forum ble det ønsket en mulighet å bruke tooltip sammen med Group Title i skjemafelter, dette fungerer slik jeg har skjønt det ikke så godt hverken i kode eller design i dag? Vi tar en kikk.

🔗 Praktiske lenker (om tilgjengelig)

Pull request: #3251
Designskisser: Components → feat: group-title/tooltip
Forhåndsvisning: Preview-link fra @fremtind-bot (om tilgjengelig)
Discussion: Link til discussion (om tilgjengelig)

🎯 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.

@onordbo onordbo added the ✨ enhancement Forslag til nye funksjoner og endringer label Nov 1, 2022
@wkillerud
Copy link
Contributor

Mulig vi bør utforske det å introdusere tooltip-teksten som en egen string prop i kode, sånn at vi ikke åpner for hva som helst av input sånn uten videre (ReactNode er veldig fleksibel).

@onordbo
Copy link
Contributor Author

onordbo commented Jan 5, 2023

Her er et løsningsforslag i Figma. Vi får løst det ganske greit via Component Properties, men for å få spacet ikonet korrekt blir tradeoffen at bredden på kompoenten fremstår litt unaturlig "ut av boksen." Fordelen med dette er at vi ivaretar auto-layout innstillingene for tekstlaget, halverer antall varianter, og samtidig tilrettelegger for å justere tooltip. I tillegg slipper vi en masse utilsiktede tooltips i eksisterende skisser, ettersom vi skjuler tooltip'en som standardinnstilling.

@onordbo onordbo self-assigned this Jan 5, 2023
@onordbo
Copy link
Contributor Author

onordbo commented Jan 5, 2023

Tar gjerne noen innspill fra @Murstam, @Stakset og @AudunBerdal på denne, siden dere såvidt meg kjent bruker Group Title ganske ofte? Om vi nå (muligens) bygger ut Group Title med flere funksjoner og den begynner å likne mer og mer på et mønster kan vi samtidig se om vi får reintrodusert spacingen per tittelstørrelse (4/8/16).

@Murstam
Copy link
Contributor

Murstam commented Jan 10, 2023

Fungerer fint så vidt jeg kan se. Bra å skjule tooltip, og ja til reintrodusere spacing 😅

@onordbo
Copy link
Contributor Author

onordbo commented Jan 16, 2023

Group Title i Figma
Her er et forslag på hvordan vi kan løse det i Figma. Ved å gjøre det på denne måten vil vi også få Tooltip i en rekke andre komponenter "gratis", siden de allerede bruker Group Title. Samtidig gjeninnfører vi innebygget spacing i komponenten så det ikke blir for trangt mellom Tooltip-ikonet og eventuelle input-felter.

NB: Tekst i Auto Layout
Jeg har lagt opp til at teksten er satt til fill når man bruker Group Title for seg selv, slik at teksten brekker over flere linjer av seg selv om den blir for lang. Men når den brukes via andre komponenter (f.eks: Text Input) tror jeg det er lurt å overstyre fra fill til hug så man slipper å endre det hver gang man skal ha tooltip som følger teksten i et inputfelt. Nedsiden ved dette blir at teksten ikke vil brekke over flere linjer (uten overstyring) hvis man har veldig lange labels i feltene. Tanker?

Variants og standardinnstillinger
Utover dette så tenker jeg at vi burde beholde både label (group title) og hjelpeteksten skjult/avslått ut av boksen, så vi unngår å knekke en god del skisser som allerede bruker "bare" selve inputfeltet.

cc @Murstam @Stakset @AudunBerdal

@onordbo
Copy link
Contributor Author

onordbo commented Feb 13, 2023

Nå har forslaget ovenfor ligget brakk lenge nok til at vi går videre med forslaget, og publiserer endringene for Group Title i Figma. Det betyr at vi reintroduserer innebygget spacing, samtidig som vi får støtte for Tooltip alle steder hvor Group Title brukes i dag.

cc @fremtind/design

@onordbo
Copy link
Contributor Author

onordbo commented Feb 15, 2023

Merget og publisert i Figma.

@onordbo onordbo closed this as completed Feb 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ enhancement Forslag til nye funksjoner og endringer
Projects
Archived in project
Development

No branches or pull requests

3 participants