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

Add NOTE clarifying presentation URL scheme support #96

Merged
merged 2 commits into from
Jun 1, 2015

Conversation

markafoltz
Copy link
Contributor

Updates spec for proposed resolution of Issue #76.

@tidoust
Copy link
Member

tidoust commented May 28, 2015

The sentence "User agents must accept a presentationUrl using the http or https schemes" sounds very normative to me, which looks strange in a note that is informative-only by definition.

I suggest to rewrite to:

<p class="note">The <code>presentationUrl</code> should name a resource accessible
to the local or a remote user agent. This specification defines behavior for
<code>presentationUrl</code> using the <code>http</code> or <code>https</code>
schemes; behavior for other schemes is not defined by this specification.</p>

Note that user agents will de facto be required to support http and https schemes because algorithms will mandate behavior in such cases. As such, there should be no need to add an extra normative statement such as "user agents MUST support presentationUrl using the http or https schemes".

The "should" in "The presentationUrl should name a resource accessible" could also be read as normative but it does not apply to user agents, so less of a problem.

@louaybassbouss
Copy link
Contributor

@mfoltzgoogle the suggestion from @tidoust LGTM

@markafoltz
Copy link
Contributor Author

Thanks @tidoust and @louaybassbouss. I've updated the PR with the suggested text and will now merge.

markafoltz added a commit that referenced this pull request Jun 1, 2015
Add NOTE clarifying presentation URL scheme support
@markafoltz markafoltz merged commit 84b8545 into gh-pages Jun 1, 2015
@markafoltz markafoltz deleted the url-schemes-issue-76 branch June 1, 2015 20:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants