mapbox / mapbox/mapbox-maps-ios
QuickZoomGestureHandler in Mapbox and Apple Maps: conflicting behaviour
Nobody has claimed this yet.
- Dominant language
- Swift
- Stars
- 601
- Forks
- 196
- PR merge metrics
- No merged PRs in 30d
Description
## New Feature
The QuickZoomGestureRecognizer allows to double-tap-pan up/down to zoom in or out on.
It would be great if there was an option to exchange the up/down gestures to zoom in/out.
## Why
These pan up/down gestures are exactly reversed to the gestures of the native Apple Maps App under iOS. Therefore this can cause some confusion for users who are used to the Appe Maps gestures. Users who want to zoom in (as they are used to in Apple Maps) do actually zoom out in Mapbox and vice versa. This might can be especially confusing if an App allows the switch between Apple Maps and Mapbox Maps.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with the QuickZoomGestureRecognizer implementation and trace how double-tap pan direction is converted into zooming. Define how an option should exchange the up/down gestures, then verify that both directions produce the expected zoom behavior on iOS. The issue does not name a file or test, so locating the relevant implementation and coverage is part of the work.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- swift
- Domain
- mobile-dev
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 45/100