docs icon indicating copy to clipboard operation
docs copied to clipboard

Published 20 hours ago • verified publisher icon pulumi

Improve mobile performance

Open joeduffy opened this issue 5 years ago • 10 comments

I didn't seen an issue already tracking this, so filing to make sure we don't lose track. We currently have relatively poor performance on mobile, as scored by Google:

image

This is hurting our SEO and ads performance.

Some specific ideas from @zchase to help move the needle here are:

  • Lazy load images
  • Take advantage of hugo's image resize api to properly size images when served on mobile
  • Figure out what to do with the unused css issue. We use tailwindcss library and I am assuming we have to load the entire library on each page

I'm assigning to both @cnunciato and @zchase since I'm unsure who technically owns this.

joeduffy avatar Feb 05 '20 02:02 joeduffy

Ran some speed experiments today and here are the results:

pageSpeedResults

Based on those results and I think the simplest path forward is lazy loading and using PurgeCSS to reduce our CSS bundle size. Though happy to hear other suggestions as well

zchase avatar Feb 06 '20 00:02 zchase

Also note HTML minification was just merged (with https://github.com/pulumi/docs/pull/2456), which reduces page sizes by ~20%.

cnunciato avatar Feb 18 '20 20:02 cnunciato

PurgeCSS work has landed (https://github.com/pulumi/docs/pull/2444). We saw a small increase in page speed but not the same increase as I was seeing locally. I will re-run the page speed test tomorrow AM and update the issue with the new metrics.

zchase avatar Feb 18 '20 23:02 zchase

Updated page speed score after @cnunciato & my changes. Progress 💪

pagespeed

zchase avatar Feb 19 '20 17:02 zchase

FWIW - this appears to have gone back down again.

Screen Shot 2020-11-13 at 10 15 48 PM

lukehoban avatar Nov 14 '20 06:11 lukehoban

slightly up again performance

susanev avatar Feb 23 '22 20:02 susanev

@kimberleyamackenzie would you have interest in investigating performance on mobile?

susanev avatar Feb 23 '22 20:02 susanev

@susanev yes, for sure!

kimberleyamackenzie avatar Feb 24 '22 00:02 kimberleyamackenzie

@cnunciato @zchase i dropped yalls assignments, but if either of you start actively working on this before the user experience team picks it up pls reassign yourselves!

susanev avatar Feb 24 '22 00:02 susanev

Thanks much. @kimberleyamackenzie note that we run Lighthouse on the website once a day as well (https://github.com/pulumi/docs/actions/workflows/check-lighthouse.yml), and we store the results of previous runs as GHA artifacts, so you should be able to see how things have changed at various URLs over time.

cnunciato avatar Feb 24 '22 01:02 cnunciato