googlearchive / vrview

Library for embedding immersive media into traditional websites.
http://developers.google.com/cardboard/vrview
Apache License 2.0
1.71k stars 1.09k forks source link

Change order of rotation to yaw and then pitch #256

Closed hasandogu closed 6 years ago

hasandogu commented 6 years ago

THREE is using local axis to rotate using Euler and the order is 'XYZ' by default. Before this change, it would rotate around X first for "pitch" which would make local Y is than world Y, once it rotates around local "Y" for yaw the hotspot would end up being in the wrong place. This is not visible in the hotspots example as no pitch is set on any of the hotspots.

googlebot commented 6 years ago

Thanks for your pull request. It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

:memo: Please visit https://cla.developers.google.com/ to sign.

Once you've signed, please reply here (e.g. I signed it!) and we'll verify. Thanks.


hasandogu commented 6 years ago

I signed it!

googlebot commented 6 years ago

CLAs look good, thanks!

lincolnfrog commented 6 years ago

Thanks for the fix!