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

Ambiguous reference to [=Window=] #1910

Closed
emlun opened this issue Jun 28, 2023 · 0 comments · Fixed by #1911
Closed

Ambiguous reference to [=Window=] #1910

emlun opened this issue Jun 28, 2023 · 0 comments · Fixed by #1911

Comments

@emlun
Copy link
Member

emlun commented Jun 28, 2023

Bikeshed currently (on commit 6dfbdba) shows this error:

LINE ~3513: Multiple possible 'window' dfn refs.
Arbitrarily chose https://drafts.csswg.org/css-color-3/#window
To auto-select one of the following refs, insert one of these lines into a <pre class=link-defaults> block:
spec:css-color-3; type:dfn; text:window
spec:fetch; type:dfn; text:window
spec:screen-capture; type:dfn; text:window
[=Window=]

The spec text in question is:

The visibility and focus state of the Window object determines whether the [[Create]](origin, options, sameOriginWithAncestors) and [[DiscoverFromExternalSource]](origin, options, sameOriginWithAncestors) operations should continue. When the Window object associated with the Document loses focus, [[Create]](origin, options, sameOriginWithAncestors) and [[DiscoverFromExternalSource]](origin, options, sameOriginWithAncestors) operations SHOULD be aborted.

And indeed, this quote from the current state of the editor's draft incorrectly links to "Window" in the CSS spec. L2 and L1 both link to "Window" in the Fetch spec, which also seems incorrect. It seems to me like this should probably point to either the Window interface or its window property, both in the HTML spec.

There is also a [=Document=] link in the same paragraph, which links to the documents concept in the DOM spec. I'm not sure if that ought to link to the Document interface too, but both are right next to each other either way.

Proposed Change

Change the [=Window=] links to {{Window}}, thus linking to the Window interface in the HTML spec.

There are references throughout the DOM spec such as "Each Window object has an associated current event", linking to the Window interface. So it seems like {{Window}} object is the preferred reference format, rather than linking to the window property.

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

Successfully merging a pull request may close this issue.

1 participant