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

Remove static like definitions of attributes and methods #235

Merged
merged 4 commits into from Oct 1, 2015

Conversation

adam-be
Copy link
Member

@adam-be adam-be commented Aug 27, 2015

Fixed for MediaStream and MediaStreamTrack

@stefhak
Copy link
Contributor

stefhak commented Aug 27, 2015

LGTM.

@adam-be
Copy link
Member Author

adam-be commented Sep 3, 2015

Pushed an additional commit that removes wording like MediaDevices.getUserMedia().

@stefhak
Copy link
Contributor

stefhak commented Sep 23, 2015

Does anyone have a problem with this PR? If not I think we should merge it. As I read it the changes are only editorial.

@@ -449,7 +449,7 @@
to that string. A good practice is to use a UUID [[rfc4122]], which
is 36 characters long in its canonical form.</p>

<p>The <dfn id="dom-mediastream-id"><code>id</code></dfn> attribute
<p>The <dfn data-lt-noDefault id="dom-mediastream-id"><code>id</code></dfn> attribute
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fyi, there's another 'id' in the paragraph above this one that you missed (the two now look different). I also wonder if this could be rewritten to avoid the double mention (let alone single mention).

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Are you referring to the id link in the paragraph above? Since it's a link, and the highlighted text is the definition, they need to look different.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(whoops, missed some comments. Thanks github and outdated diff!)

Makes sense now. I was probably expecting this paragraph (with definition) to come before the other.

@adam-be
Copy link
Member Author

adam-be commented Sep 30, 2015

@jan-ivar, can you take a look at the latest changes?

@jan-ivar
Copy link
Member

lgtm!

alvestrand added a commit that referenced this pull request Oct 1, 2015
Remove static like definitions of attributes and methods
@alvestrand alvestrand merged commit 2299190 into w3c:master Oct 1, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants