Going from AFrame 1.3.0 to 1.4.1 broke rotation on pinch #90

Closed
opened 1 year ago by utopiah · 2 comments
utopiah commented 1 year ago
Owner

In fact was probably already broken by going from 1.3 to 1.4 thus why it's been pinned until now.

Might be related to API change https://github.com/aframevr/aframe/issues/5185

Should check the rotation in the pinchmoved event of the princhprimary component e.g https://git.benetou.fr/utopiah/text-code-xr-engine/src/branch/code-editor/index.html#L1054

PS: also bumps threejs from ^0.137.0 to ^0.147.1

In fact was probably already broken by going from 1.3 to 1.4 thus why it's been pinned until now. Might be related to API change https://github.com/aframevr/aframe/issues/5185 Should check the rotation in the `pinchmoved` event of the `princhprimary` component e.g https://git.benetou.fr/utopiah/text-code-xr-engine/src/branch/code-editor/index.html#L1054 PS: also bumps threejs from ^0.137.0 to ^0.147.1
Poster
Owner

Fixed via 4cf9c0b148 as indeed the API changed with element.rotation.x becoming element.rotation._x (same for y and z).

Fixed via https://git.benetou.fr/utopiah/text-code-xr-engine/commit/4cf9c0b1485d199c853128ffa75abe92a97c16a1 as indeed the API changed with `element.rotation.x` becoming `element.rotation._x` (same for y and z).
utopiah closed this issue 1 year ago
Poster
Owner

Actually fixed by 699dbcd6b9 using the proper finger name. Stranger that somehow it did work before few times.

Actually fixed by https://git.benetou.fr/utopiah/text-code-xr-engine/commit/699dbcd6b9f00de4059829d6e24a6e566b12b757 using the proper finger name. Stranger that somehow it did work before few times.
Sign in to join this conversation.
No Milestone
No project
No Assignees
1 Participants
Notifications
Due Date

No due date set.

Dependencies

No dependencies set.

Reference: utopiah/text-code-xr-engine#90
Loading…
There is no content yet.