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

[ios] Limit annotation view pan gesture check to own recognizer [SUPERSEDED] #5812

Closed
wants to merge 1 commit into from

Conversation

friedbunny
Copy link
Contributor

@friedbunny friedbunny commented Jul 27, 2016

UPDATE: Closed this PR in favor of #5813, which was filed against the release-ios-v3.3.0 branch for inclusion in v3.3.2.

Allows non-dragging pans that are initiated on views to pan the underlying map.

Annotation views each have long press and pan gesture recognizers. If a
view is not draggable or not in a positive dragging state, the pan
gesture recognizer should not begin. This commit makes the criteria for
rejecting pans more strict — only `_panGestureRecognizer` should be
rejected, not the entire `UIPanGestureRecognizer` class.
@friedbunny friedbunny added bug iOS Mapbox Maps SDK for iOS annotations Annotations on iOS and macOS or markers on Android labels Jul 27, 2016
@friedbunny friedbunny added this to the ios-v3.3.2 milestone Jul 27, 2016
@friedbunny friedbunny self-assigned this Jul 27, 2016
@mention-bot
Copy link

@friedbunny, thanks for your PR! By analyzing this pull request, we identified @boundsj and @1ec5 to be potential reviewers.

@friedbunny friedbunny closed this Jul 27, 2016
@friedbunny friedbunny deleted the fb-pan-on-annotation-views branch July 27, 2016 23:35
@friedbunny friedbunny removed this from the ios-v3.3.2 milestone Jul 27, 2016
@friedbunny friedbunny changed the title [ios] Limit annotation view pan gesture check to own recognizer [ios] Limit annotation view pan gesture check to own recognizer [SUPERSEDED] Jul 27, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
annotations Annotations on iOS and macOS or markers on Android bug iOS Mapbox Maps SDK for iOS
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants