-
Notifications
You must be signed in to change notification settings - Fork 38
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]: <Stepper /> #2509
Comments
Vi endret vekten på teksten for en stund siden til kun regular fordi bold førte til layout shift (CLS). Jeg er redd dette forslaget vil resultere i enda større layout shift. Har dere testet å bruke tabs? Eller lagd en egen komponent for fremdrift i "saker"? |
Takk for oppfølger. Vet ikke når dere endret til kun Regular men både vertical og horizontal stepper har kun Strong title på samtlige steps i min versjon fra i Core UI Kit og det er ingen updates tilgjengelig. At alt er Regular og ikke Strong (SemiBold) er bedre enn at alt er Strong, men jeg syns fortsatt at title burde følge vekting på sirklene for active og completed step. Mitt innspill var forøvrig todelt, det ene går på typografisk vekting av stegene og det andre på størrelsesvarianter. Har dere vurdert Small og/eller en text only versjon? Vi forsøker å holde egne komponenter til et minimum men lager dem selvsagt om det er nødvendig, vi har ikke laget en egen stepper løsning eller annet fremdriftskomponent. Jeg føler tabs har en annen bruksrolle enn stepper forøvrig, så jeg har ikke tenkt på det som noe substitutt. Progresjonshandlinger er forøvrig ingen utbredt sak i vårt miljø men jeg syns generelt det er enklere å ta initiativ til å designe løsninger for det om ready-made komponentene fra Aksel er justert for Small da det som kjent gjør de mer plasseringvennlige i tight UI etc 😃 |
Ja, my bad. Vi endret så all tekst er strong. |
Ja, small kan vi se på. En "liggende" variant av horisontal stepper vil nok fortsatt ha behov for nummerikonet for å ikke trigge layout shift. Det vil bygge ut bredden ganske mye (ikon + tekst), og gjøre den veldig sårbar for endring i bredde. |
Innspill til Small versjon av horizontal. (også nevnt på Slack)
Reduser title label fra Body/Short/Large til Body/Short/Medium. Behold Body/Short/LargeStrong for Number. Det vil i alle fall for horizontal versjonen av stepper bli litt mer plasseringsvennlig for interne flater. Legger også ved forslag til text only versjon som reduserer høyden den okkuperer. Min Figma skisse gir 24px containerhøyde. Flyttet completed ikonet til høyre for label og redusert størrelsen på den til 24x24px.
Jeg syns uavhengig av Small eller Medium også at kun active state burde ha Strong title - inactive burde ha label i regular vekt, slik det feks allerede er på Toggle group.
The text was updated successfully, but these errors were encountered: