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

components: Rewrite Spinner for better performance #2384

Merged
merged 2 commits into from
Jan 23, 2023

Conversation

lachlanjc
Copy link
Member

Closes #1547. I'm not sure how to do the same performance testing the author of that issue did to compare it, but reportedly SMIL is more performant.

Credit to respinner for this implementation idea!

@lachlanjc lachlanjc linked an issue Jan 16, 2023 that may be closed by this pull request
@vercel
Copy link

vercel bot commented Jan 16, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated
theme-ui ✅ Ready (Inspect) Visit Preview Jan 16, 2023 at 7:25AM (UTC)

@codesandbox-ci
Copy link

codesandbox-ci bot commented Jan 16, 2023

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit 716501e:

Sandbox Source
next-theme-ui-example Configuration
gatsby-plugin-theme-ui-example Configuration

@lachlanjc lachlanjc added affects-docs Changes affect documentation, but not only documentation @theme-ui/components labels Jan 16, 2023
Copy link
Member

@hasparus hasparus left a comment

Choose a reason for hiding this comment

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

Nice one!

@lachlanjc lachlanjc merged commit 2d8bfc3 into develop Jan 23, 2023
@lachlanjc lachlanjc deleted the 1547-improve-spinner-performance branch January 23, 2023 20:39
@hasparus hasparus added the prerelease This change is available in a prerelease. label Feb 7, 2023
@hasparus hasparus mentioned this pull request Feb 7, 2023
@hasparus
Copy link
Member

hasparus commented Feb 7, 2023

🚀 PR was released in v0.15.5 🚀

@hasparus hasparus added released This issue/pull request has been released. and removed prerelease This change is available in a prerelease. labels Feb 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
affects-docs Changes affect documentation, but not only documentation released This issue/pull request has been released. @theme-ui/components
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Improve Spinner performance
2 participants