zoomSpeed is overwritten after first mousewheel event

[z1ndabad]

Describe the bug
After changing the value of controls().zoomSpeed, the initial mousewheel event zooms at the specified speed, but future events received by the underlying globe DOM node zoom at the default speed.

useEffect(() => {
    const controls = globeRef.current?.controls();
    if (controls) {
      controls.zoomSpeed = 10;
    }
  }, [globeRef.current]);

Reproduction
https://codesandbox.io/p/sandbox/react-globe-gl-forked-f89hnq?workspaceId=ff4eac7f-570c-4bb7-aa7a-be79c7cbdde4

• Zoom in on the globe once -- zoom speed is fast
• Zoom in on the globe again -- zoom is slow
• Reload page, zoom in on the globe -- zoom speed is fast again

Expected behavior
Setting controls().zoomSpeed should override the zoom speed for the lifetime of the DOM node.

Desktop

• OS: Windows
• Browser: Chrome 128.0.6613.114

Additional context & Workaround
The zoom speed scaling based on altitude may be responsible: https://github.com/vasturiano/globe.gl/blob/2a8a6f39717a9fd213782f943de2ce557045f699/src/globe.js#L392

I worked around this by adding a new event listener, but I'm not sure how to exactly reproduce the zoomSpeed variation based on altitude (i.e. properly setting POV from outside the package). Happy to attempt a PR if you have feedback on where to put the user-supplied speed in the zoom-adjustment function.

  useEffect(() => {
    const controls = globeRef.current.controls();
    const ZOOM_SPEED = 2;
    if (globeRef.current && controls) {
      controls.zoomSpeed = ZOOM_SPEED;

      // Workaround
      controls.addEventListener("change", () => {
        let pov = globeRef.current.pointOfView();
        controls.rotateSpeed = pov.altitude * 0.2;
        controls.zoomSpeed = (pov.altitude + 1) * ZOOM_SPEED;
      });
    }
  }, [globeRef]);

[vasturiano]

@z1ndabad thanks for reaching out.

You can override that built-in behaviour of the progressive zoom speed by tapping into the onZoom event handler.

For example:

<Globe
  ref={globeRef}
  ...
  onZoom=(useCallback(() => globeRef.current.controls().zoomSpeed = 10, [globeRef]))
/>

That will override the default behaviour with a constant zoom speed of 10.