Skip to content

Update to reference SVG 2#3692

Merged
annevk merged 3 commits intowhatwg:masterfrom
dstorey:reference-svg2
May 18, 2018
Merged

Update to reference SVG 2#3692
annevk merged 3 commits intowhatwg:masterfrom
dstorey:reference-svg2

Conversation

@dstorey
Copy link
Copy Markdown
Member

@dstorey dstorey commented May 16, 2018

Remove reference to SVG 1.1 and SVG Tiny
@dstorey
Copy link
Copy Markdown
Member Author

dstorey commented May 16, 2018

I liked to the SVG draft as the current CR hasn't been updated yet with all the removals mentioned in the HTML spec. I wasn't sure if this is appropriate, but we can always update once we publish the next PR/CR

Copy link
Copy Markdown
Member

@domenic domenic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A few nits around the introductory material, but the hard parts look great.

We actually always use the editor's drafts in the references, so even if there were a CR or other TR, we wouldn't want to reference it.

source Outdated
<li>The <code data-x="">tabindex</code> attribute is allowed on visible SVG elements</li>
<li>The ARIA accessibility attributes are allowed on all SVG elements</li>
</ul>
<p>User agents that implement SVG must not implement the SVG 2 specification</p>
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This sentence doesn't seem quite right :). Maybe must implement the <cite>SVG 2</cite> specification, and not any earlier revisions? And don't forget the period ending the paragraph.

source Outdated
</ul>
<p>User agents that implement SVG must not implement the SVG 2 specification</p>

<p>The following features are defined in the SVG specifications:</p>
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Now that there's only one, this doesn't need to be plural.

@dstorey
Copy link
Copy Markdown
Member Author

dstorey commented May 16, 2018

I think this one is ready now?

Copy link
Copy Markdown
Member

@domenic domenic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Oh, did you want to add yourself the acknowledgments section?

@dstorey
Copy link
Copy Markdown
Member Author

dstorey commented May 17, 2018

I think I'm already there now.

@annevk annevk merged commit 7069de4 into whatwg:master May 18, 2018
@annevk
Copy link
Copy Markdown
Member

annevk commented May 18, 2018

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants