maxZoom?: number instead of maxZoom?: number | null

[Volper212]

In the docs it says that maxZoom can be null for infinite scaling. However, the type of maxZoom when one hovers over it is number | undefined, not number | null | undefined.

(property) maxZoom?: number | undefined

[elliottkember]

I think the docs just need updating, then. Sounds as though it used to take null but no longer requires the prop. Would you mind making a PR for this change?

[Volper212]

null actually works as the docs say (it just does not pass the type check with TypeScript's strict mode):

react-native-zoomable-view/src/ReactNativeZoomableView.tsx

Line 604 in 34342f0

if (maxZoom !== null && newZoomLevel > maxZoom) {

So not the docs but the types need to be updated. Alternatively, one could get rid the option of passing null altogether and just use maxZoom={Infinity} for infinite zoom. There are two options then.