- From: Anssi Kostiainen via GitHub <sysbot+gh@w3.org>
- Date: Thu, 25 Feb 2016 06:46:41 +0000
- To: public-media-capture@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