Added zoom control

[scarlac]

• Added zoom control
• Added max zoom control
• Added onZoom handler

const [zoom, setZoom] = useState(0);
<Camera
  zoomMode="on"
  zoom={zoom}
  maxZoom={15}
  torchMode={torchMode ? 'on' : 'off'}
  onZoom={(e) => {
    console.log('zoom', e.nativeEvent.zoom);
    setZoom(e.nativeEvent.zoom);
  }}
/>

[scarlac]

@DavidBertet in case you're interested.

[DavidBertet]

@DavidBertet in case you're interested.

Looking at it!

[DavidBertet]

Should that be done by the native side / Camera component? Sounds like the expected behavior as a user of the framework. RealCamera probably does it already in update(cameraType

[scarlac]

Since it uses the declarative API it must fully control the zoom factor. Thus, it must also manually reset the value.
The only alternative to these things are to use an imperative API for zooming which circumvents the issues. It all boils down to the fact that the gesture control and default zoom are only available in native

[DavidBertet]

We can use defaultProps bellow to set 20 by default. We might want it on Android as well

Suggested change

	<NativeCamera maxZoom={props.maxZoom ?? 20} style={{ minWidth: 100, minHeight: 100 }} ref={nativeRef} {...props} />
	);
	});
	Camera.defaultProps = {
	<NativeCamera style={{ minWidth: 100, minHeight: 100 }} ref={nativeRef} {...props} />
	);
	});
	Camera.defaultProps = {
	maxZoom: 20,

[scarlac]

Good catch. I actually wanted to remove that. maxZoom shouldn't be required in RN, as that means we can't tell if it's undefined (unrestricted) or 20 (restricted).

[DavidBertet]

not "iOS only" anymore

[elliottkember]

I think this may have reverted a change made in https://github.com/teslamotors/react-native-camera-kit/pull/551/files?w=1

[scarlac]

Thanks @elliottkember - This was re-fixed and released in v14.0.0-beta13.