search

redwoodjs / redwoodjs.com

redwoodjs.com redesign for 1.0
21 stars 23 forks source link

Do not inline startup logos #47

Closed pavelloz closed 2 years ago

pavelloz commented 2 years ago

This little change makes initial html page ~30KB smaller by not inlining images that are 3 screens below the fold (read: are not critical to be ASAP). Should help speed the page a little bit.

netlify[bot] commented 2 years ago

Deploy Preview for redwood-sprout ready!

Name Link
Latest commit e23f8612b4d7db0dcc6e6dc032348525b4c9962b
Latest deploy log https://app.netlify.com/sites/redwood-sprout/deploys/62616656c5d5030008a76fbd
Deploy Preview https://deploy-preview-47--redwood-sprout.netlify.app
Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

zakmandhro commented 2 years ago

Looks good. The tape.sh logo's aspect ratio looks off. @dac09 @pavelloz can we fix that before merging this?

pavelloz commented 2 years ago

Ah, missed that one. Sure :)

pavelloz commented 2 years ago

@zakmandhro should be better, and future proofed in case some other sponsors have logos in dimensions that do not fit into previous constrains.