W3C home > Mailing lists > Public > public-media-capture@w3.org > February 2016

[mediacapture-depth] Pull Request: Fix #8: Define depthMapUnit constrainable property per extensibility guidelines

From: Anssi Kostiainen via GitHub <sysbot+gh@w3.org>
Date: Tue, 23 Feb 2016 16:37:55 +0000
To: public-media-capture@w3.org
Message-ID: <pull_request.opened-60368883-1456245472-sysbot+gh@w3.org>
anssiko has just submitted a new pull request for 
https://github.com/w3c/mediacapture-depth:

== Fix #8: Define depthMapUnit constrainable property per 
extensibility guidelines ==
@burnburn This is an attempt to (re)spec a new `depthMapUnit` 
constrainable property (specific to mediacapture-depth) per the 
[Defining a new constrainable property][1] extensibility guidelines 
(added in [#244][2]). 

Given you authored the said guidelines, I think you'd be in the best 
position to help review this PR. From the editor's point of view, the 
large number of partial dictionaries that need to be added in order to
 spec a new constrainable property seemed high. For example, partial 
`MediaTrackConstraintSet, MediaTrackSupportedConstraints, 
MediaTrackCapabilities` dictionaries looked boilerplate-ish to me, so 
I am wondering whether you have suggestions how to streamline this.

Probably the easier way to review is to use the [RawGit preview of the
 spec with these changes][3]. Thanks in advance!


[1]: 
https://w3c.github.io/mediacapture-main/#defining-a-new-constrainable-property
[2]: https://github.com/w3c/mediacapture-main/issues/244
[3]: 
https://rawgit.com/w3c/mediacapture-depth/issue-8-depthmapunit/index.html#depthmapunit-constrainable-property

See https://github.com/w3c/mediacapture-depth/pull/97
Received on Tuesday, 23 February 2016 16:37:57 UTC

This archive was generated by hypermail 2.3.1 : Tuesday, 23 February 2016 16:37:57 UTC