Skip to content
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 document.all as having [[IsHTMLDDA]] #3087

Merged
merged 2 commits into from
Oct 2, 2017
Merged

Conversation

TimothyGu
Copy link
Member

Fixes #3015.

One less willful violation!

Copy link
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.

Oh, nice, thank you! Some minor style nits, but otherwise LGTM.

source Outdated
@@ -3056,6 +3056,7 @@ a.setAttribute('href', 'https://example.com/'); // change the content attribute
<li>The <dfn data-x-href="https://tc39.github.io/ecma262/#sec-sharedarraybuffer-objects"><code>SharedArrayBuffer</code></dfn> class</li>
<li>The <dfn data-x-href="https://tc39.github.io/ecma262/#sec-native-error-types-used-in-this-standard-typeerror"><code>TypeError</code></dfn> class</li>
<li>The <dfn data-x="js-RangeError" data-x-href="https://tc39.github.io/ecma262/#sec-native-error-types-used-in-this-standard-rangeerror"><code>RangeError</code></dfn> class</li>
<li>The <dfn data-x="js-IsHTMLDDA" data-x-href="https://tc39.github.io/ecma262/#sec-IsHTMLDDA-internal-slot">[[IsHTMLDDA]]</dfn> internal slot</li>
Copy link
Member

Choose a reason for hiding this comment

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

We've kind of stoppped adding data-x's for most JS things. This makes them a bit easier to link to.

source Outdated
<span data-x="js-IsHTMLDDA">[[IsHTMLDDA]]</span> internal slot.</p>

<div class="note">

Copy link
Member

Choose a reason for hiding this comment

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

We've also kind of moved away from extra newlines between opening/closing tags and their contents. So here and for the ul, best to remove those.

@TimothyGu
Copy link
Member Author

@domenic Done!

@annevk annevk merged commit 542be55 into whatwg:master Oct 2, 2017
@annevk
Copy link
Member

annevk commented Oct 2, 2017

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