Dolly to Cursor and focalOffset about camera-controls

[AgentCodeBanks]

I had the same problem with my project. I solved it by recalculating the target for the controller with every mouse-up event (after every rotation with changing the orbitPoint). I scale the look vector of the camera with the radius of the spherical in the control and then move the new target with moveTo. Similar to the fitToSphere

const spherical = new THREE.Spherical(0, 0, 0);
const vTempA = new THREE.Vector3();
const vTempB = new THREE.Vector3();
		
container.addEventListener('mouseup', (evt) => {
	if(event.which === 3 ){
		if (controls._camera.isOrthographicCamera) {
			controls.getSpherical(spherical); //Get sperical who is used to calculate the camera position in the update loop
			vTempA.set(controls._camera.position.x, controls._camera.position.y, controls._camera.position.z); //Get camera position
			vTempB.set(0, 0, -1).applyQuaternion(controls._camera.quaternion); //Get the look vector from the camera
			vTempB.multiplyScalar(spherical.radius); //scale the look vector
			vTempB.add(vTempA); // add the camera position to get the new target position

			controls.setFocalOffset(0, 0, 0, false).then(); 
			controls.moveTo(vTempB.x, vTempB.y, vTempB.z, false).then(); //Move the new target to the position
			controls.update(0.0001);
		} else {
			controls.setOrbitPoint(0, 0, 0, false);
		}
	}	
});

I haven't checked the other functions of the controller to see if they still work, as that was the most important thing for me that time.

#yomotsu Do you see any possible problems with the code?

[yomotsu]

Thanks for using camera-controls.
I think it's because setOrbitPoint calls setFocalOffset internally, and focalOffset causes many problems, like

• #303
• #316

TBH, I would like to remove focalOffset and related features such as setOrbitPoint, but still available for backward compat😢

The best solution is, to avoid using setFocalOffset...

[DmitriyIudokhin]

But in your examples setOrbitPoint with dollyToCursor is working well. Maybe it is because of using Perspective camera(your examples) instead of Orthographic(in my app)?

[yomotsu]

setOrbitPoint with dollyToCursor

Come to think of it, that's right.
Would you mind providing a simplified working demo using codesanbox or so to just reproduce your problem?

setFocalOffset will break dollyToCursor = true anyway...