Skip to content
This repository has been archived by the owner on Aug 8, 2023. It is now read-only.

Zooming gestures problem on 5.1.4 #10082

Closed
Petrulak opened this issue Sep 28, 2017 · 2 comments
Closed

Zooming gestures problem on 5.1.4 #10082

Petrulak opened this issue Sep 28, 2017 · 2 comments
Labels
Android Mapbox Maps SDK for Android

Comments

@Petrulak
Copy link

Petrulak commented Sep 28, 2017

Platform:Android
Mapbox SDK version:5.1.4

Steps to trigger behavior

  1. Add mapbox 5.1.4 map into your view. Use following settings.
        mapBoxMap.uiSettings.isRotateGesturesEnabled = false
        mapBoxMap.uiSettings.isLogoEnabled = false
        mapBoxMap.uiSettings.isAttributionEnabled = false
        mapBoxMap.uiSettings.isTiltGesturesEnabled = false
  1. Run the app
  2. Start zooming with 2 fingers.

Expected behavior

The map should zoom in/out.

Actual behavior

The map doesn't zoom in/out. Once it zooms in out, it's not smooth at all.
Seems like it jumps from one frame to second one.
Please see attached video
Device on the video is One plus 5
mapbox_zooming_5_1_4.mp4.zip

@tobrun tobrun added the Android Mapbox Maps SDK for Android label Sep 28, 2017
@tobrun
Copy link
Member

tobrun commented Sep 28, 2017

@Petrulak performance improvements (using GLSurfaceView) is coming with the 5.2.0 release.
With that in place we will be hitting 60 fps. Would you be able to the 5.2.0-SNAPSHOT for a test? In the next few weeks we are releasing a couple of beta versions and would love some feedback on it before a final 5.2.0 release.

To use snapshots, add the snapshots repo in the root build.gradle file:

allprojects {
    repositories {
        jcenter()
        maven { url "http://oss.sonatype.org/content/repositories/snapshots/" }
    }
}

In you app build.gradle replace current dependency with:

    compile(com.mapbox.mapboxsdk:mapbox-android-sdk:5.2.0-SNAPSHOT@aar) {
        transitive = true
    }

Going to close this as the ticket tracking general improvements to gestures can be found in #10016.

@tobrun tobrun closed this as completed Sep 28, 2017
@Petrulak
Copy link
Author

@tobrun is there any timeline for #10016 ?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Android Mapbox Maps SDK for Android
Projects
None yet
Development

No branches or pull requests

2 participants