Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork928
fix(webapp): remove Spline 3D animation to reduce bundle size by ~47%#2763
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
base:main
Are you sure you want to change the base?
Uh oh!
There was an error while loading.Please reload this page.
Conversation
|
coderabbitaibot commentedDec 7, 2025 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
WalkthroughThe pull request removes Spline-based animated background functionality from the ErrorDisplay component. The changes include: removing Framer Motion and Spline imports from ErrorDisplay.tsx, deleting the animated spline scene JSX block, simplifying the container and wrapper div layouts from relative positioning with z-index and margin offsets to a cleaner flex layout structure, and removing the "@splinetool/react-spline" dependency from package.json. The component's core content (Header, message, button) and props remain unchanged. Estimated code review effort🎯 2 (Simple) | ⏱️ ~8 minutes
Pre-merge checks and finishing touches❌ Failed checks (1 warning)
✅ Passed checks (2 passed)
✨ Finishing touches
🧪 Generate unit tests (beta)
Thanks for usingCodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
beydogan commentedDec 11, 2025
If you are fine with this change but don't like the empty state of the page, if you provide the logo asset, I can try to do something with lottie or css that looks something similar visually. |
Uh oh!
There was an error while loading.Please reload this page.
I noticed the dashboard takes a while(20-30seconds) to load sometimes and that led me investigate the reason. I found out one of the JS chunks is 2035kb, which is almost half of the whole JS chunks.
That chunk is three.js library. Which is a dependency for
@splinetool/react-spline, which is only being used to render a spline scene on the error page.As a user, I prefer to have a fast loading dashboard instead of a fancy 3D animation on an error page. Though, same/similar animation can be achieved via lottie or as css animation probably, which will have much smaller size.
This change basicallyreduces JS bundle size from 4283kb to 2248kb. Even though the assets are being cached I get this slow loading dashboard several times per day. I find that a bit poor UX.
Further optimization is possible by implementing tree shaking tocompanyicons to bundle only used icons to the build instead of all the icons. That would probably cut another ~600kb sincewebapp is using only 5 icons from there but loading whole 748kb.
Also noticed the assets are being served without gzip on production, which would improve the situation if you want to keep the animation.
I'm noticing this issue since currently I'm in Türkiye and have a high latency to US-East. Probably a CDN would help a lot too!
✅ Checklist
Testing
Ran locally and went to a 404 page.
Changelog
fix(webapp): remove Spline 3D animation to reduce bundle size
Screenshots
Before
After
Resources
💯