Fix cross-spec links and references #174
Conversation
Minor editorial updates to fix a few broken links and correct references: - Drop custom definitions that are no longer needed - Replace "triggered by user activation" with new user activation model (see whatwg/html#5129) - Use back ticks for values instead of var shorthand - Drop link to feature name in Feature Policy (term no longer exists)
index.bs
Outdated
| @@ -501,7 +491,7 @@ This specification defines a <a>policy-controlled feature</a> that controls | |||
| whether the <a>request Picture-in-Picture algorithm</a> may return a | |||
| {{SecurityError}} and whether {{pictureInPictureEnabled}} is `true` or `false`. | |||
|
|
|||
| The <a>feature name</a> for this feature is `"picture-in-picture"`. | |||
| The feature name for this feature is `"picture-in-picture"`. | |||
There was a problem hiding this comment.
If "feature name" doesn't exist anymore, shall we update spec text that says "If the document is not allowed to use the policy-controlled feature named "picture-in-picture", throw a SecurityError and abort these steps." as well?
There was a problem hiding this comment.
Actually, to avoid the dubious repetition "the feature name for this feature is", I just pushed an update to merge the sentence here with the previous paragraph .
Even though the term "feature name" is no longer defined, it still seems reasonable to talk about the name of a feature, so I didn't try to update the algorithm step. If you prefer, we could perhaps simply drop "named" in both cases to end up with 'policy-controlled feature "picture-in-picture"'.
There was a problem hiding this comment.
f4b4902 looks good to me.
Minor editorial updates to fix a few broken links and correct references:
Preview | Diff