-
Notifications
You must be signed in to change notification settings - Fork 1.6k
Conversation
Hello! This is an automated review by our custom reviewbot. It updates automatically when code or GitHub comments in this pull request are created or updated. Requested changesIf there are any common problems with the content files you created or modified, they will be listed here.
|
✔️ Deploy Preview for web-dev-staging ready! 🔨 Explore the source changes: b445f20 🔍 Inspect the deploy log: https://app.netlify.com/sites/web-dev-staging/deploys/61c5060bed7eff0008e4c921 😎 Browse the preview: https://deploy-preview-7092--web-dev-staging.netlify.app |
* update videos and images, some text * added missing alt text * image replacements and more adjustments
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.
Mostly nits for grammar / style guide.
Plus, 👏🏻 👏🏻 👏🏻 for the screen reader coverage.
Co-authored-by: Alexandra White <alexandrawhite@google.com>
This is now failing the linter, I'll figure it out tomorrow unless @heyawhite can spot it. |
I’ll check in an hour or so and see if I can spot it.
|
@rachelandrew it was an HTML character problem -- it's been resolved. Good to go! |
Fixes: #6783
Post for Dec 28.