Skip to content

refactor(nextjs-app-v14): DSW-3178 Added freeze for animation for spinner and chip#352

Open
JoshuaNg2332 wants to merge 1 commit intomainfrom
DSW-3178-updated-animation-freeze-nextjs-14
Open

refactor(nextjs-app-v14): DSW-3178 Added freeze for animation for spinner and chip#352
JoshuaNg2332 wants to merge 1 commit intomainfrom
DSW-3178-updated-animation-freeze-nextjs-14

Conversation

@JoshuaNg2332
Copy link
Copy Markdown
Contributor

No description provided.

{ url: '/components/checkbox', name: 'Checkbox' },
{ url: '/components/checkbox-group', name: 'Checkbox Group' },
{ url: '/components/chip', name: 'Chip' },
{ url: '/components/chip', name: 'Chip', pauseBeforeScreenshot: true },
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm unsure if this pause will be enough to prevent each spinner screenshot being slightly different.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah I was thinking that as well. Do you reckon it would need to be longer?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants