Skip to content

Conversation

abernier
Copy link
Collaborator

@abernier abernier commented Aug 14, 2025

Caution

DO NOT MERGE as is

Warning

see ⚠️ in review: some changes are fork's specific

this PR shows the work of my fork abernier/camera-controls

@abernier abernier self-assigned this Aug 14, 2025
Comment on lines -40 to -62
let THREE: THREESubset;
let _ORIGIN: _THREE.Vector3;
let _AXIS_Y: _THREE.Vector3;
let _AXIS_Z: _THREE.Vector3;
let _v2: _THREE.Vector2;
let _v3A: _THREE.Vector3;
let _v3B: _THREE.Vector3;
let _v3C: _THREE.Vector3;
let _cameraDirection: _THREE.Vector3;
let _xColumn: _THREE.Vector3;
let _yColumn: _THREE.Vector3;
let _zColumn: _THREE.Vector3;
let _deltaTarget: _THREE.Vector3;
let _deltaOffset: _THREE.Vector3;
let _sphericalA: _THREE.Spherical;
let _sphericalB: _THREE.Spherical;
let _box3A: _THREE.Box3;
let _box3B: _THREE.Box3;
let _sphere: _THREE.Sphere;
let _quaternionA: _THREE.Quaternion;
let _quaternionB: _THREE.Quaternion;
let _rotationMatrix: _THREE.Matrix4;
let _raycaster: _THREE.Raycaster;
Copy link
Collaborator Author

@abernier abernier Aug 14, 2025

Choose a reason for hiding this comment

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

now static and accessible through this.ctor.* (more portable when subclassing)

@@ -1,6 +1,9 @@
{
"name": "camera-controls",
"name": "@abernier/camera-controls",
Copy link
Collaborator Author

@abernier abernier Aug 14, 2025

Choose a reason for hiding this comment

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

⚠️ fork specific

"dist"
],
"repository": "yomotsu/camera-controls",
"repository": "abernier/camera-controls",
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

⚠️ fork specific

Comment on lines +3 to +5
- name: canary-*
prerelease: true
channel: canary
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

support for canary release for canary-* branches

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

chromatic integration ❤️

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

i've exported more useful stuff

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

added dependabot

* setup

* ci

* .

* .
* CameraControlsState["spherical"]

* add a test

* simpler test
Comment on lines +1 to +7
[![Latest NPM release](https://img.shields.io/npm/v/@abernier/camera-controls.svg)](https://www.npmjs.com/package/@abernier/camera-controls)
[![Storybook dev branch](https://img.shields.io/badge/dev-f7f9fc.svg?logo=storybook)](https://dev--68888af2a4f99a664aba1eb5.chromatic.com)
[![Chromatic dev branch](https://img.shields.io/badge/dev-171c23.svg?logo=chromatic)](https://www.chromatic.com/library?appId=68888af2a4f99a664aba1eb5&branch=dev)

this is my fork of original [yomotsu/camera-controls](https://github.com/yomotsu/camera-controls)

<details>
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

⚠️ fork specific

@yomotsu
Copy link
Owner

yomotsu commented Sep 2, 2025

@abernier
thank you for implementing these features. Looking forward to it, but no rush.

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