Skip to content

Implemented scrollDuringRotateOrZoom control for Google Maps#3784

Merged
christopherdro merged 3 commits intoreact-native-maps:masterfrom
lukasz-app:scrollDuringRotateOrZoom
Apr 20, 2021
Merged

Implemented scrollDuringRotateOrZoom control for Google Maps#3784
christopherdro merged 3 commits intoreact-native-maps:masterfrom
lukasz-app:scrollDuringRotateOrZoom

Conversation

@lukasz-app
Copy link
Contributor

Does any other open PR do the same thing?

No

What issue is this PR fixing?

This PR exposes the native functionality of GoogleMaps to enable/disable moving the map while rotating or zooming.
The default behavior is to allow this, but I needed to disable it in my project.

This PR might fix:
#2572

How did you test this PR?

I tested this in the example project and in another one as well, by linking it in package.json to my fork of this library.
I tested both platforms, on physical devices

Steps to test:

  1. Include my fork in package.json in your project
    "react-native-maps": "https://github.com/lukaszchopin/react-native-maps.git#scrollDuringRotateOrZoom"
  2. Check whether changing scrollDuringRotateOrZoomEnabled to false disables possibility to move map while rotating or zooming:
    <MapView scrollDuringRotateOrZoomEnabled={false} />

@christopherdro christopherdro merged commit ebb13b5 into react-native-maps:master Apr 20, 2021
@LaurenceM10NS
Copy link

Same issue here with latest version. I tried in android only.

borjomemost pushed a commit to borjomemost/react-native-maps that referenced this pull request May 6, 2022
…ative-maps#3784)

* implemented scrollDuringRotateOrZoom control

* fix lint issue

* note in docs
pinpong pushed a commit to pinpong/react-native-maps that referenced this pull request Feb 28, 2025
…ative-maps#3784)

* implemented scrollDuringRotateOrZoom control

* fix lint issue

* note in docs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants