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

Conversation

@kimstacy
Copy link
Contributor

Fixes typo prefers-constrast to prefers-contrast in 🇰🇷 What's New in DevTools (Chrome 96)

Changes proposed in this pull request:

  • Change prefers-constrastprefers-contrast in article and description

I noticed this typo also appears in the Chrome DevTools Twitter graphic. As I'm unauthorized to post a corrected version, I'd like to propose an amended repost by the account maintainer.

Graphic Typo

Thank you!

@google-cla google-cla bot added the cla: yes label Nov 18, 2021
@netlify
Copy link

netlify bot commented Nov 18, 2021

✔️ Deploy Preview for developer-chrome-com ready!

🔨 Explore the source changes: 4352c3c

🔍 Inspect the deploy log: https://app.netlify.com/sites/developer-chrome-com/deploys/6195c5bdfaa72600075a0799

😎 Browse the preview: https://deploy-preview-1744--developer-chrome-com.netlify.app

@chrome-devrel-review-bot
Copy link
Collaborator

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 changes

If there are any common problems with the content files you created or modified, they will be listed here.

site/_includes/partials/devtools/ko/whats-new.md

  • Please put a blank line before and after each section heading. Some Markdown parsers don't detect the text as a section heading unless there is blank space before and after it. Affected lines: 1, 4, 19

site/ko/blog/new-in-devtools-96/index.md

  • Please put a blank line before and after each section heading. Some Markdown parsers don't detect the text as a section heading unless there is blank space before and after it. Affected lines: 50, 69, 70

1 similar comment
@chrome-devrel-review-bot
Copy link
Collaborator

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 changes

If there are any common problems with the content files you created or modified, they will be listed here.

site/_includes/partials/devtools/ko/whats-new.md

  • Please put a blank line before and after each section heading. Some Markdown parsers don't detect the text as a section heading unless there is blank space before and after it. Affected lines: 1, 4, 19

site/ko/blog/new-in-devtools-96/index.md

  • Please put a blank line before and after each section heading. Some Markdown parsers don't detect the text as a section heading unless there is blank space before and after it. Affected lines: 50, 69, 70

@kimstacy
Copy link
Contributor Author

Sorry about the double Markdown commits. I'm not immune to typos myself! :)

CC @TORU0239 @cwdoh 리뷰 부탁드립니다. 감사합니다!

@cwdoh
Copy link
Contributor

cwdoh commented Nov 19, 2021

Good point! Thanks for fixing it.:)

@mihajlija
Copy link
Contributor

@chybie Has this typo in the image been fixed? I don't see it in the post.

@kimstacy
Copy link
Contributor Author

@cwdoh You’re welcome! I’m happy to help. Thank you and @TORU0239 @chybie @mihajlija for your hard work!

@jecfish
Copy link
Contributor

jecfish commented Dec 1, 2021

Thanks all!

@jecfish jecfish merged commit 7afd63f into GoogleChrome:main Dec 1, 2021
@jecfish
Copy link
Contributor

jecfish commented Dec 1, 2021

The image isn't use in the post, but in Twitter (no edit function). Merge this PR for the typo fix. Thanks!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants