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

[ios, macos] Fix an issue where a wrong annotation may selected if annotations were set close together #11438

Merged
merged 4 commits into from
Mar 13, 2018

Conversation

lloydsheng
Copy link
Contributor

@lloydsheng lloydsheng commented Mar 13, 2018

A copy of PR #11168 and added fix for macOS.

@lloydsheng lloydsheng requested review from 1ec5 and friedbunny March 13, 2018 03:34
Copy link
Contributor

@1ec5 1ec5 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!


// The first selection in the cycle should be the one nearest to the
// tap. Also the annotation tags need to be stable in order to compare them with
// the remembered tags _annotationsNearbyLastTap.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nit: the variable is named _annotationsNearbyLastClick.

@1ec5 1ec5 merged commit 4404a26 into release-boba Mar 13, 2018
@1ec5 1ec5 deleted the lloyd-annotation-select branch March 13, 2018 18:26
@friedbunny friedbunny added iOS Mapbox Maps SDK for iOS annotations Annotations on iOS and macOS or markers on Android labels Mar 13, 2018
@friedbunny friedbunny added this to the ios-v4.0.0 milestone Mar 13, 2018
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 iOS Mapbox Maps SDK for iOS
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants