Re: [mediacapture-depth] Split dictionary members

Note this PR is very much WIP, so I would be cautious to review it too carefully. Based on a quick scan of https://w3c.github.io/mediacapture-image/ there are no overlapping constraints per naming at least. Ping also @astojilj to confirm the details.

We're trying to figure out how to best spec the extensions to constrains pattern while keeping the spec concise and clear. Inheriting the boilerplate directly from mediacapture-main was our initial attempt but led to a spec that was hard to follow (and edit).

For example, in mediacapture-main, the following "Dictionary Foo Members" sections duplicate information already in the IDL definition, and add "See Baz for details." indirections that eventually all link to the same "Constrainable Properties" table:

https://www.w3.org/TR/mediacapture-streams/#idl-def-MediaTrackSupportedConstraints
https://www.w3.org/TR/mediacapture-streams/#idl-def-MediaTrackCapabilities
https://www.w3.org/TR/mediacapture-streams/#idl-def-MediaTrackConstraintSet
https://www.w3.org/TR/mediacapture-streams/#idl-def-MediaTrackSettings

Our plan is to link directly from the dictionary IDL members to the "Constrainable properties" section where we define the expected behaviour for each dictionary member per kind (color or depth). Probably we should get rid of the table as well since it is bad for readability, making the columns too narrow for text. Also "Notes" column contains normative assertions which is confusing.

I like the format in https://w3c.github.io/mediacapture-image/#constrainable-properties better over https://www.w3.org/TR/mediacapture-streams/#constrainable-properties, so we probably do something similar for this spec.

Recently we've been busy with the [Blink][1] implementation and have experimented with some [demos][2] using our implementation. We'll follow up with spec updates baking in our implementation feedback (including the editorial ones discussed above) hopefully soon.

[1]: https://crbug.com/616098
[2]: https://github.com/01org/depthcamera-pointcloud-web-demo

-- 
GitHub Notification of comment by anssiko
Please view or discuss this issue at https://github.com/w3c/mediacapture-depth/pull/158#issuecomment-288340307 using your GitHub account

Received on Wednesday, 22 March 2017 09:16:22 UTC