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
Update class/ID/tagname docs #6302
Conversation
|
||
Internal AMP class names prefixed with `-amp-` and `i-amp-` are disallowed in AMP HTML. | ||
|
||
Consult AMP documentation for meaning of class names prefixed `amp-`. The use of these classes is allowed and meant to allow customization of some features of AMP runtime and extensions. |
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.
prefixed with amp-
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.
Done
|
||
Consult AMP documentation for specific extensions before using `amp-` and `AMP` IDs to avoid conflict with the features provided by these extensions, such as `amp-access`. | ||
|
||
All other authored class names are allowed in AMP HTML markup. |
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.
s/class names/IDs
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.
Done
LGTM—thanks! |
* Update class/ID/tagname docs * review fixes
* Update class/ID/tagname docs * review fixes
No description provided.