-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
HTML shouldn't <dfn> Document #1715
Comments
"Document" links in the HTML spec itself should be internal links, not to the DOM spec. |
Can't we just mark it as partial interface somehow? |
This is another case for "some way to opt a DFN out of Bikeshed processing". I'll go ahead and add that today. (partials are handled by Bikeshed at the parsing level - if it notices that you're defining a partial interface, it forces the interface name to be an A, rather than choosing A/DFN based on whether there's already an instance of that interface defined elsewhere. There's nothing at the definition-contract level that indicates something is a partial definition.) |
Done. Setting |
@tabatkins though if it parses it as partial, it should be clear too it cannot be "the" |
Right, that's what I'm saying - in a Bikeshed document, Bikeshed knows about partial interfaces, and prevents them from becoming definitions by default. But there's nothing in the data model that indicates an interface definition came from a partial block, so if a non-Bikeshed spec has |
…TML. DOM already defines Document. Fixes whatwg#1715
DOM already defines Document. Other specs that cross-reference Document should link to DOM, not HTML's partial interface. The data-dfn-lt="" attribute hides the definition for Bikeshed. Fixes whatwg#1715.
A Bikeshed file containing
{{Document}}
complains that:HTML's is only a partial interface, so it shouldn't use markup that overrides DOM's definition for other specs. I'm not sure if the right thing to do is to remove the
<dfn>
tag, mark it withdata-noexport=""
, or something else. @tabatkins, can you advise?The text was updated successfully, but these errors were encountered: