-
Notifications
You must be signed in to change notification settings - Fork 60
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
Task: Improve performance of Find Properties page #854
Comments
Can I be assigned this please. |
assigned! |
I minified the logo svg, but it didn't much improve performance. Interestingly, when I ran the lighthouse report (which I did in an incognito browser with Chrome as well) I got significantly different results compared to #734. The largest contentful paint wasn't that slow to begin with for me, but the total blocking time was much worse, mostly due to main-thread-work, particularly script evaluation. Do you have suggestions for what to try next? For the record I only included screenshots of the report after minifying the logo (using vecta.io) because there was not much of a difference in the scores. Find Properties Desktop Lighthouse ReportDesktop Main Thread Work DetailsFind Properties Mobile Lighthouse ReportMobile Main Thread Work Details |
@CodeWritingCow |
Hi @AZBL, what is your minified SVG's file size vs the current SVG on the production site? If there's a significant reduction without loss of image quality, can you open a PR to replace the SVG in prod with your minified version? Thanks! We can run a Lighthouse audit and check if we'd improve performance by using the minified file in prod. |
Describe the task
Clean and Green Philly's Find Properties page has a Google Lighthouse performance score of 58/100 on mobile devices, and 60/100 on desktop devices. The page's
Largest Contenful Paint
is taking 6.3 seconds, and the website's logo (https://www.cleanandgreenphilly.org/logo.svg) appears to be causing this.Reduce this page's image loading time and Improve the score to high 80s or low-to-mid 90s.
Acceptance Criteria
Additional context
The text was updated successfully, but these errors were encountered: