-
Notifications
You must be signed in to change notification settings - Fork 12
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
KOMP-217-progress #398
KOMP-217-progress #398
Conversation
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Jeg ser du har feilende tester. Det skyldes at wcag krever at dette elementet blir brukt sammen med en label. I dokumentasjonen er vi jo opptatt av å vise selve elementet, og ikke labelen, men ta gjerne en titt på slider for å se på hvordan det er løst der.
), | ||
}; | ||
|
||
export const ProgressAnimated: Story = { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Du kan kanskje putte en isAnimated også under animert, men bortsett fra det ser det helt glimrende ut 😁
Yes! Det hender jeg lager PRer uten at jeg har sjekket testene, men det fikses alltid😎 Jeg kan sørge for at jeg ikke lager PR før jeg har sjekket, så dere slipper å se på koden før den er helt ferdig😅 |
Beskrivelse
Legger til dokumentasjon for Progress.
Sjekkliste