Skip to content

Spec: Document Gamepad integration#553

Merged
toji merged 5 commits into
masterfrom
spec-gamepad
Apr 4, 2019
Merged

Spec: Document Gamepad integration#553
toji merged 5 commits into
masterfrom
spec-gamepad

Conversation

@toji

@toji toji commented Mar 4, 2019

Copy link
Copy Markdown
Member

Work-in-progress PR for gamepad integration spec text. Mostly still need to address the gamepad.id formatting. It's far enough along though that some feedback would be useful. I'd especially like feedback on the new mapping diagram and whether it helps or hurts understanding.

@toji
toji requested a review from NellWaliczek March 4, 2019 21:02
@toji toji added this to the Next Working Draft milestone Mar 4, 2019
Comment thread index.bs Outdated
Comment thread index.bs Outdated
Comment thread index.bs
Comment thread index.bs
Comment thread index.bs Outdated
Comment thread index.bs Outdated
Comment thread index.bs Outdated
Comment thread index.bs Outdated
@toji
toji force-pushed the spec-gamepad branch 2 times, most recently from 34d97ee to f4e292a Compare March 12, 2019 22:57
@toji
toji marked this pull request as ready for review March 12, 2019 22:58
@toji

toji commented Mar 12, 2019

Copy link
Copy Markdown
Member Author

Updated to address Nell's comments. PTAL!

@toji

toji commented Mar 13, 2019

Copy link
Copy Markdown
Member Author

Okay, added a requirement for gripSpace before you can use xr-standard. (Also fixed a minor thing that was bugging me about the mapping diagram.) One more look?

@toji

toji commented Apr 3, 2019

Copy link
Copy Markdown
Member Author

Review ping for @NellWaliczek. Any further changes you'd like to see? I'm feeling pretty good that this is ready-to-land.

@NellWaliczek NellWaliczek left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for your thorough work on this one!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants