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
amp-mathml: few fixes and validator tests #13001
Conversation
}); | ||
}); | ||
|
||
// set data-formula so 3P script case use it. |
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.
Typo: "can use it".
}); | ||
|
||
// set data-formula so 3P script case use it. | ||
this.element.setAttribute('data-formula', |
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.
This is a bit confusing. Why have the extra formula
attribute if you're just going to use it to set the data-formula
attribute and not just use data-formula
? Basically what happens if the developer sets the data-formula
attribute? We should at least make this clear in the doc.
* Revision bump for ampproject#12862 * Revision bump for ampproject#13001 * Revision bump for ampproject#12906
* Revision bump for ampproject#12862 * Revision bump for ampproject#13001 * Revision bump for ampproject#12906
* Revision bump for ampproject#12862 * Revision bump for ampproject#13001 * Revision bump for ampproject#12906
inline
attribute to the documentation