Skip to content

Conversation

@mkungla
Copy link
Contributor

@mkungla mkungla commented Aug 4, 2016

closes #1717

@ngokevin
Copy link
Member

ngokevin commented Aug 4, 2016

Thanks! Looks good. Waiting for tests to pass.

@mkungla
Copy link
Contributor Author

mkungla commented Aug 4, 2016

yeah travis had false alarm there due (network timeout) and is now re running the tests.

@ngokevin
Copy link
Member

ngokevin commented Aug 4, 2016

Yup, I restarted it.

registerPrimitive('a-camera', {
defaultComponents: {
camera: {},
camera: {
Copy link
Member

Choose a reason for hiding this comment

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

one liner here?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@dmarcos across other primitives single values are not

@dmarcos dmarcos merged commit f026198 into aframevr:master Aug 4, 2016
@mkungla mkungla deleted the userheight-16 branch August 4, 2016 02:01
JChehe added a commit to JChehe/aframe that referenced this pull request Aug 18, 2016
CHANGELOG.md says "Default camera is now positioned at 0, 1.6, 0 rather than 0, 1.8, -4. In VR mode, the 1.6m height offset as defined by camera.userHeight is removed. (aframevr#1474, aframevr#1718)"

but in v0.3 doc, still v0.2 value of Default camera position.

My English is so poor~Sorry
@JChehe JChehe mentioned this pull request Aug 18, 2016
ngokevin pushed a commit that referenced this pull request Aug 18, 2016
CHANGELOG.md says "Default camera is now positioned at 0, 1.6, 0 rather than 0, 1.8, -4. In VR mode, the 1.6m height offset as defined by camera.userHeight is removed. (#1474, #1718)"

but in v0.3 doc, still v0.2 value of Default camera position.

My English is so poor~Sorry
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Consider lowering default userHeight a tad.

3 participants