search

mdn / sprints

Archived: MDN Web Docs issues are tracked in the content repository.
https://github.com/mdn/content
Creative Commons Zero v1.0 Universal
150 stars 142 forks source link

Specifications link for ShadowRoot.delegatesFocus should be removed #3917

Closed unarist closed 3 years ago

unarist commented 3 years ago

Request type

Details

https://wiki.developer.mozilla.org/en-US/docs/Web/API/ShadowRoot/delegatesFocus

This article refers to https://dom.spec.whatwg.org/#shadowroot-delegates-focus but the spec only mentions about an internal field, not a public attribute on interfaces. So the link should be removed I think.

sideshowbarker commented 3 years ago

cc @gregwhitworth, who added the spec link (to place in the spec where the internal field is defined)

For the record here, in Blink ShadowRoot objects do expose a delegatesFocus attribute — because the Blink IDL at https://chromium.googlesource.com/chromium/src/+/refs/heads/master/third_party/blink/renderer/core/dom/shadow_root.idl has it. But it’s not clear why — since no such property is defined in the IDL at https://dom.spec.whatwg.org/#interface-shadowroot for ShadowRoot, nor defined in the interface chain ShadowRoot inherits from (Node/DocumentFragment)

So I’ve raised https://github.com/mdn/browser-compat-data/pull/7620 to mark this as non-standard in BCD.

chrisdavidmills commented 3 years ago

I think I've fixed the spec link too: https://github.com/mdn/content/commit/196fb5d90f7551993e729a2a9fd422e2f482eacd