-
Notifications
You must be signed in to change notification settings - Fork 40
Copy editing of non-normative text #429
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
Conversation
@@ -3308,10 +3312,16 @@ | |||
Kanai, Tobie Langel, Tomoyuki Shimizu, Travis Leithead, and Wayne Carr | |||
for help with editing, reviews and feedback to this draft. | |||
</p> | |||
<p> | |||
<em>AirPlay</em>, <em>HDMI</em>, <em>Chromecast</em>, <em>DLNA</em> and |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@tidoust, is the recommendation to explicitly call these out in the spec, or does the "Non-W3C Trademarks; Member Trademarks" section in https://www.w3.org/Consortium/Legal/2002/trademarks-20021231 cover specs as well? I assume the term "Site" in this generic boilerplate refers to any resource hosted on the w3.org domain, including specs in the TR space.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I checked internally. Specs we publish usually do not contain non-W3C trademarks for various reasons. In our case, mentioning these technologies as examples in the introduction is important to clarify the context for readers, and the introduction is clear that other technologies are available.
It is a good idea to call these trademarks out explicitly in the acknowledgement section. The proposed text is fine. I recommend adding "They are only cited as background information" to the end of it to make it crystal clear that one does not need them to implement the specification.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done.
index.html
Outdated
uses the operating system to send the resulting graphical and audio | ||
output to the presentation display. We refer to this situation as the | ||
<dfn data-lt="1-UA">1-UA mode</dfn> implementation of the Presentation | ||
API. The only requirement is that the user agent be able to send |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
s/be able to/is able to/
Thanks @mfoltzgoogle, great work. Before landing, I'd like @tidoust to clarify how we should handle the non-W3C trademarks. |
I made another pass to polish the introduction and the Cross Origin Access section. I'm pretty happy now :) To improve legibility, I updated to link the first time a term like presentation appears in a given paragraph, and not link following occurrences. |
@mfoltzgoogle, @anssiko, see minor proposed tweak to ack section inline. That looks good to me otherwise. |
Addresses Issue #345: Introduction could use come copy editing.
This updates the prose in the introduction to use more precise language and improve style.
It makes minor editing improvements to non-normative notes and the security and privacy section.
It also adds acknowledgements of registered trademarks used in the specification.
Preview | Diff