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

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

From: Anssi Kostiainen via GitHub <sysbot+gh@w3.org>
Date: Thu, 25 Feb 2016 06:46:41 +0000
To: public-media-capture@w3.org
Message-ID: <pull_request.closed-60368883-1456382799-sysbot+gh@w3.org>
anssiko has just merged pull request 97 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 Thursday, 25 February 2016 06:46:43 UTC

This archive was generated by hypermail 2.3.1 : Thursday, 25 February 2016 06:46:44 UTC