Add ForceLoadAtTop Document Policy #131
Conversation
bokand
force-pushed
the
documentPolicyIntegration
branch
from
8def939
to
192986f
Compare
|
@clelland: I added some spec text for the ForceLoadAtTop document policy, does this look about right? The document policy spec mentions there's a registry of features but I can't find one. If there is one we could add this to it. |
index.bs
Outdated
| navigating-to-text-fragment section --> | ||
|
|
||
| This specification defines a <a href="https://w3c.github.io/webappsec-permissions-policy/document-policy.html#configuration-point">configuration point</a> | ||
| in [[!document-policy|Document Policy]] with name "force-load-at-top". It is a |
There was a problem hiding this comment.
I think we would say that it "has" a type of "boolean" (or just that "its type is boolean")
index.bs
Outdated
| "status": "ED", | ||
| "publisher": "W3C", | ||
| "deliveredBy": [ | ||
| "https://w3c.github.io/webappsec-permissions-policy" |
There was a problem hiding this comment.
It probably doesn't matter for an inline biblio ref, but this should be deliveredby "https://www.w3.org/2011/webappsec/" (URL points to the WG)
There was a problem hiding this comment.
Thanks, done. (btw, let me know if there's a way to reference the existing spec - I couldn't get it to work and I assume it's because Document-Policy isn't in specref yet.)
index.bs
Outdated
| This otherwise case is the same as the current step 3.3. | ||
| </div> | ||
| 3. <a href="https://w3c.github.io/webappsec-permissions-policy/document-policy.html#algo-get-policy-value">Get | ||
| the document policy value</a> of "force-load-at-top" for the |
There was a problem hiding this comment.
This can just be "Get the policy value for force-load-at-top in the Document"
|
This looks good -- just a couple places where I would have worded things differently, but nothing big. Re: registries, I think you'd be the first entry, so I should probably get on adding that :) |
|
Thanks! Let me know when you get the registry up and we can add this to it. |
Fixes #80
Preview | Diff