Closed AdamGleave closed 1 month ago
Name | Link |
---|---|
Latest commit | 547d149e6ce96eaf6a79cb9b2e0e7ca2848d889a |
Latest deploy log | https://app.netlify.com/sites/goattack/deploys/664b041f94a262000836cf8b |
Deploy Preview | https://deploy-preview-111--goattack.netlify.app |
Preview on mobile | Toggle QR Code...Use your smartphone camera to open QR code link. |
Lighthouse |
1 paths audited Performance: 98 (no change from production) Accessibility: 97 (no change from production) Best Practices: 100 (no change from production) SEO: 92 (š¢ up 7 from production) PWA: - View the detailed breakdown and full score reports |
To edit notification comments on pull requests, go to your Netlify site configuration.
Performance:
wgo.min.js
may need to be loaded beforewgo.player.js
.esbuild
. This only saved ~8kB though (most of the CSS is including icons, so doesn't minify well).Misc: Instructed Netlify to use latest stable Node.js version 20.x, previously it was stuck on the first version we'd used 18.x that is now deprecated. This doesn't seem to have changed anything.
SEO: added meta description tag to every page.
Before vs after
Before:![image](https://github.com/AlignmentResearch/KataGoVisualizer/assets/1547683/ed34058d-6c05-4a77-a430-6d85989bdf57)
After:
Before:
After:
We do still have quite slow "First Contentful Paint" but I don't see obvious low-hanging fruit here -- it's probably hard to do well here when your whole application is in JS... Maybe we could remove unused components of Bootstrap to reduce that load time.