Controls: Require element in connect()

[mrdoob]

Description

Finally found some time to implement #29079 (comment).

This PR changes this recent API (1):

const controls = new THREE.OrbitControls( camera );
controls.domElement = element;
controls.connect();

To this (2):

const controls = new THREE.OrbitControls( camera );
controls.connect( element );

This API still works (3):

const controls = new THREE.OrbitControls( camera, element );

@Mugen87 Not sure if there's a way of not breaking the (1) version?

[github-actions]

📦 Bundle size

Full ESM build, minified and gzipped.

	Before	After	Diff
WebGL	336.08 78.29	336.08 78.29	+0 B +0 B
WebGPU	525.21 146.28	525.21 146.28	+0 B +0 B
WebGPU Nodes	524.67 146.18	524.67 146.18	+0 B +0 B

🌳 Bundle size after tree-shaking

Minimal build including a renderer, camera, empty scene, and dependencies.

	Before	After	Diff
WebGL	465.15 112.18	465.15 112.18	+0 B +0 B
WebGPU	598.01 162.57	598.01 162.57	+0 B +0 B
WebGPU Nodes	553.1 152.04	553.1 152.04	+0 B +0 B

[Mugen87]

Not sure if there's a way of not breaking the (1) version?

I don't think so. But given the new warning I think the migration task is obvious.

I've also added the change to the migration guide for r175.