-
Notifications
You must be signed in to change notification settings - Fork 131
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
Define script element processing model #196
Comments
Question to the WG: What exactly do we need to specify? |
We probably need to find the exact algorithms & definitions in HTML to include by reference. |
see also the PR on html by @anne |
The SVG Working Group just discussed The full IRC log of that discussion<AmeliaBR> Topic: Define script element processing model<AmeliaBR> github: https://github.com//issues/196 <AmeliaBR> chris: domenic already said (in an HTML PR) what needs to be done. We can copy the HTML edits. <AmeliaBR> … I think we should get something in the spec & then ask them to review. <AmeliaBR> krit: I think the changes are (1) include the new mixin, and (2) reference the right section. <AmeliaBR> … anyone willing to do the edits? <AmeliaBR> chris: I could take a try, but not super familiar with the topic. <AmeliaBR> Amelia: OK. You can always bring the issue back up if anything is unclear. <AmeliaBR> … Do we need a resolution? <AmeliaBR> krit: Wait until we have a proposed text. <AmeliaBR> chris: The bits about currentScript, is that implemented or defined anywhere? <AmeliaBR> Amelia: I think it's defined in HTML, just need to acknowledge that that can be an SVG script. |
SVG 2 has no details on how the script element is processed.
document.currentScript may return an SVGScriptElement, but we don't define how that is set.
See whatwg/html#913
Need to do some analysis to determine what else is missing.
Ideally we would defer as much as possible to the HTML spec. I think this is intended by the statement currently in the SVG spec that says:
But that's not enough.
The text was updated successfully, but these errors were encountered: