Skip to content
This repository has been archived by the owner on Mar 14, 2024. It is now read-only.

I/O 2019: Speed at Scale blog post recap #931

Merged
merged 13 commits into from
May 23, 2019
Merged

I/O 2019: Speed at Scale blog post recap #931

merged 13 commits into from
May 23, 2019

Conversation

addyosmani
Copy link
Member

This is a post summarizing three takeaways from the Speed at Scale talk @khempenius and I gave at Google I/O. As most of our talk focused on case studies, we're going to do this first round-up and follow up later with individual write-ups as needed.

This pull request adds a new:

  • Guide(s) 📘
  • Codelab(s) 🔬

Have you read the wiki on writing a guide or a codelab?

  • Done! 🍰

Does your content reference any Glitch projects?
Have you read the wiki on working with Glitch?
Have you added the Glitch project to our team page?

  • My changes do not contain any Glitches. 🙅‍♂️
  • I have read the wiki on working with Glitch. 🍰
  • I have added my Glitches to the team page. 👏

src/site/_data/contributors.json Outdated Show resolved Hide resolved
@Meggin Meggin self-requested a review May 23, 2019 15:34
src/site/content/en/blog/speed-at-scale/index.md Outdated Show resolved Hide resolved
src/site/content/en/blog/speed-at-scale/index.md Outdated Show resolved Hide resolved
src/site/content/en/blog/speed-at-scale/index.md Outdated Show resolved Hide resolved

During the "Speed at Scale" [talk](https://www.youtube.com/watch?v=YJGCZCaIZkQ&feature=youtu.be) at Google I/O 2019 we announced three things that we hope will improve web performance over the coming year.

<iframe width="560" height="315" src="https://www.youtube.com/embed/YJGCZCaIZkQ" frameborder="0" allow="accelerometer; autoplay; encrypted-media; gyroscope; picture-in-picture" allowfullscreen></iframe>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The video is a little bit lost with so much white space. This may be something to fix on infra-side but still worth calling it out.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch. I'll see if playing around with the width/height would still work for the responsive layout for this post, but can otherwise defer to infra if I can't do better :)

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You can copy what the portals folks did maybe?

<div style="width:100%; padding-top: 56.25%; position: relative;">
  <iframe style="width:100%; height: 100%;position: absolute; top: 50%; left: 50%; transform: translate(-50%,-50%);" src="https://www.youtube.com/embed/Ai4aZ9Jbsys?start=1081" frameborder="0" allow="accelerometer; autoplay; encrypted-media; gyroscope; picture-in-picture" allowfullscreen></iframe>
</div>

src/site/content/en/blog/speed-at-scale/index.md Outdated Show resolved Hide resolved
src/site/content/en/blog/speed-at-scale/index.md Outdated Show resolved Hide resolved
src/site/content/en/blog/speed-at-scale/index.md Outdated Show resolved Hide resolved
src/site/content/en/blog/speed-at-scale/index.md Outdated Show resolved Hide resolved
src/site/content/en/blog/speed-at-scale/index.md Outdated Show resolved Hide resolved

The Google Fonts documentation has been updated to include `font-display` in the default code embeds. We hope this will encourage more developers to try out this exciting addition.

<figure class="w-figure">
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What is this image showing? It's not entirely clear to me. Maybe it could be better introduced?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The image is showing the default embed code window from Google Fonts. I've tried to update the text to note that the embed we are referring to is in the image below it. Does that help?

@addyosmani addyosmani dismissed TimvdLippe’s stale review May 23, 2019 18:00

Addressed contributors issue

Copy link
Contributor

@robdodson robdodson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added a few suggestions.

If it's too much work to turn some of the items into embedded Glitches feel free to reply with "nahhhhhhh" :D

@@ -1273,4 +1273,4 @@
"github": "rowan-m",
"glitch": "rowan-m"
}
}
}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

newline plz :)


During the "Speed at Scale" [talk](https://www.youtube.com/watch?v=YJGCZCaIZkQ&feature=youtu.be) at Google I/O 2019 we announced three things that we hope will improve web performance over the coming year.

<iframe width="560" height="315" src="https://www.youtube.com/embed/YJGCZCaIZkQ" frameborder="0" allow="accelerometer; autoplay; encrypted-media; gyroscope; picture-in-picture" allowfullscreen></iframe>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You can copy what the portals folks did maybe?

<div style="width:100%; padding-top: 56.25%; position: relative;">
  <iframe style="width:100%; height: 100%;position: absolute; top: 50%; left: 50%; transform: translate(-50%,-50%);" src="https://www.youtube.com/embed/Ai4aZ9Jbsys?start=1081" frameborder="0" allow="accelerometer; autoplay; encrypted-media; gyroscope; picture-in-picture" allowfullscreen></iframe>
</div>

src/site/content/en/blog/speed-at-scale/index.md Outdated Show resolved Hide resolved
src/site/content/en/blog/speed-at-scale/index.md Outdated Show resolved Hide resolved
src/site/content/en/blog/speed-at-scale/index.md Outdated Show resolved Hide resolved
src/site/content/en/blog/speed-at-scale/index.md Outdated Show resolved Hide resolved
src/site/content/en/blog/speed-at-scale/index.md Outdated Show resolved Hide resolved
src/site/content/en/blog/speed-at-scale/index.md Outdated Show resolved Hide resolved
src/site/content/en/blog/speed-at-scale/index.md Outdated Show resolved Hide resolved
@addyosmani addyosmani requested a review from robdodson May 23, 2019 20:51
 into speed-at-scale

* 'speed-at-scale' of https://github.com/GoogleChrome/web.dev:
  Update src/site/content/en/blog/speed-at-scale/index.md
  Update src/site/content/en/blog/speed-at-scale/index.md
  Update src/site/content/en/blog/speed-at-scale/index.md
  Update src/site/content/en/blog/speed-at-scale/index.md
  Update src/site/content/en/blog/speed-at-scale/index.md
  Update LightWallet image for Speed at Scale blog post
@robdodson robdodson merged commit 33790a3 into master May 23, 2019
@robdodson robdodson deleted the speed-at-scale branch May 23, 2019 22:37
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants