-
Notifications
You must be signed in to change notification settings - Fork 22.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Convert noteblocks for mdn folder #35080
Conversation
This PR converts the noteblocks for the 'mdn' folder to GFM syntax, using a [conversion script](https://github.com/queengooborg/mdn-toolkit/blob/main/upgrade-noteblock.js).
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.
Existing changes aren't regressions so I wouldn't block, but (1) I see several notes not converted and I wonder if there are more similar cases; (2) The meta docs about callout syntax have already been updated separately, right?
@@ -155,7 157,8 @@ The following checklist is good to keep in mind while writing and reviewing cont | |||
- **Add image information**: Include proper [`alt`](/en-US/docs/Web/HTML/Element/img#alt) text on all images and diagrams. This text, as well as captions on tables and other figures, counts because spiders can't crawl images, and so `alt` text tells search engine crawlers what content the embedded media contains. | |||
> **Note:** It is not recommended to include too many keywords or keywords not related to the feature in an attempt to manipulate search engine rankings; this type of behavior is easy to spot and tends to be penalized. |
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.
Why isn't this converted?
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.
Noteblocks inside of lists are getting incorrectly formatted by Markdownlint, so I avoided converting them for the time being! (That applies to any noteblocks that are left unconverted as well!)
This PR converts the noteblocks for the 'mdn' folder to GFM syntax, using a conversion script.