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

VRLayer's leftBounds/rightBounds attributes: Sequences must not be used as the type of an attribute #166

Closed
ddorwin opened this issue Dec 20, 2016 · 1 comment
Labels
spec bug Inconsistencies in spec
Milestone

Comments

@ddorwin
Copy link
Contributor

ddorwin commented Dec 20, 2016

The spec currently has:

interface VRLayer {
  ...
  readonly **attribute sequence**<float> leftBounds;
  readonly **attribute sequence**<float> rightBounds;
};

Per https://heycam.github.io/webidl/#idl-sequence:

Sequences must not be used as the type of an attribute or constant.

Note: This restriction exists so that it is clear to specification writers and API users that sequences are copied rather than having references to them passed around. Instead of a writable attribute of a sequence type, it is suggested that a pair of operations to get and set the sequence is used.

toji added a commit that referenced this issue Jan 7, 2017
Attempts to address #142, #107, #166, #156, and #125. Feedback welcome!
@toji toji added the spec bug Inconsistencies in spec label Feb 2, 2017
@toji toji modified the milestone: 1.2 Feb 7, 2017
@toji
Copy link
Member

toji commented Mar 8, 2017

Closing since this is planned to be addressed in 2.0. If anyone has issues with the 2.0 proposal, please file a new issue. For details see the "VRCanvasLayer" interface described here: https://github.com/w3c/webvr/blob/master/explainer.md#appendix-b-proposed-idl

@toji toji closed this as completed Mar 8, 2017
@cwilso cwilso modified the milestones: Spec-Complete for 1.0, 1.0 Apr 30, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
spec bug Inconsistencies in spec
Projects
None yet
Development

No branches or pull requests

3 participants