euanwm / OpenWeightlifting

Repository that is deployed for the OpenWeightlifting site
https://www.openweightlifting.org
BSD 3-Clause "New" or "Revised" License
31 stars 21 forks source link

Decrease render time on initial load #245

Closed euanwm closed 1 month ago

euanwm commented 11 months ago

Is your feature request related to a problem? Please describe. On an initial load of the main landing page, a large amount of time is spent loading the logo within the header bar.

Describe the solution you'd like Implement lazy loading for the OWL logo so that it scores higher on PageSpeed Insights

Additional context Screenshot from 2023-10-01 09-55-12 image

inomag commented 11 months ago

Hey, can I work on this?

euanwm commented 11 months ago

@inomag sure

inomag commented 11 months ago

Hi, @euanwm Can you check this. I am not able to run the project on my system to check the update LCP score. https://github.com/euanwm/OpenWeightlifting/pull/251

rsbrowne commented 10 months ago

Hey @inomag, not sure if you're still working on this but given that the image being loaded is using next/image I would recommend simply adding the priority attribute to the Image tag (see docs here). We don't want images above the fold being lazy loaded at all.

if you're not still working on it don't worry, I can throw up a PR for this later today

euanwm commented 10 months ago

@rsbrowne - It's all yours buddyyyy.

euanwm commented 1 month ago

Linked to #388

Closing