Note how the SVG currently implemented in browsers is a mix of SVG 1.1 and SVG Tiny 1.2 #327
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…1 and SVG Tiny 1.2 Browsers do not currently follow any specifications exactly, for historical and practical reasons. This attempts to clarify what features are best avoided, and what features can be relied on. Discussion: #126
|
@heycam or @erikdahlstrom, would you be willing to review this change for accuracy? |
| @@ -37272,16 +37272,61 @@ dictionary <dfn>TrackEventInit</dfn> : <span>EventInit</span> { | |||
|
|
|||
| </div> | |||
|
|
|||
| <p>When the SVG <code data-x="">foreignObject</code> element contains elements from the <span>HTML | |||
| namespace</span>, such elements must all be <span>flow content</span>. <ref spec=SVG></p> | |||
| <p>When the SVG <code data-x="svg:foreignObject">foreignObject</code> element contains elements | |||
There was a problem hiding this comment.
If you are not referencing a <dfn>, just keep data-x set to the empty string. I don't think this change will go through the build script as-is.
Fixed `<code data-x=“”>` attributes to conform with the spec syntax.
|
Still an outstanding comment about removing the line + <p>Additionally, <code>data-*</code> attributes are allowed on all SVG elements.</p>(Since no SVG spec AFAIK allows this yet.) |
added 2 commits
November 16, 2015 12:20
It works in all browsers, but has never been specced. Better to be on the safe side.
|
@zcorpan Whoops, missed that. |
|
Lgtm |
|
Thank you @tigt! |
|
No, thank you for being so helpful |
cscott
added a commit
to fgnass/domino
that referenced
this pull request
Oct 18, 2016
Removal of contentScriptType/contentStyleType attributes: whatwg/html#327 https://www.w3.org/Bugs/Public/show_bug.cgi?id=25377 whatwg/html@c350dad Removal of externalResourcesRequired attribute: https://www.w3.org/Bugs/Public/show_bug.cgi?id=26067 whatwg/html@068eaf4
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Browsers do not currently follow any specifications exactly, for
historical and practical reasons. This attempts to clarify what
features are best avoided, and what features can be relied on.
Discussion: #126