-
Notifications
You must be signed in to change notification settings - Fork 1.6k
Add initial Layout Instability API draft #996
Conversation
|
||
Here are a few examples of how the layout shift score is calculated: | ||
|
||
![Layout shift example #1](layout-shift-1.png) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For all of the image examples, would you mind enhancing the alt text a bit? If you were describing the image to someone over the phone how would you phrase it?
}); | ||
``` | ||
|
||
<div class="w-aside w-aside--note"> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We have shortcodes you can use for this. Sorry if it's not well documented.
{% Aside 'note' %}
[you can put markdown in here]
{% endAside %}
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I wanted to customize what was bolded, and it didn't seem like I could do that with the short codes. Let me know if I'm wrong.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Left a few comments related to alt text and shortcode usage. Up to you if you feel like updating to the shortcodes.
770b635
to
ce7bc34
Compare
ce7bc34
to
a83e103
Compare
Fixes #993.
This PR is an initial draft that I'd like to get reviewed (as this is my first PR to web.dev).
I'm also waiting on some additional feedback from a few other folks, so please don't merge/publish until I give the go-ahead.