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

[6749] fix annotation tap detection (smaller area for annotation imag… #6756

Closed
wants to merge 1 commit into from

Conversation

RomainQuidet
Copy link
Contributor

@RomainQuidet RomainQuidet commented Oct 19, 2016

I know tap detection is quite sensitive, my tests show it works well on different annotation images and views sizes, with offsets.

Fixes #6749.

@mention-bot
Copy link

@RomainQuidet, thanks for your PR! By analyzing the history of the files in this pull request, we identified @boundsj, @1ec5 and @rmnblm to be potential reviewers.

@tobrun tobrun added the iOS Mapbox Maps SDK for iOS label Oct 19, 2016
@1ec5
Copy link
Contributor

1ec5 commented Oct 19, 2016

@boundsj @incanus, can you review?

@1ec5 1ec5 added the annotations Annotations on iOS and macOS or markers on Android label Oct 19, 2016
@1ec5
Copy link
Contributor

1ec5 commented Oct 19, 2016

Also, if you don’t mind, please make the corresponding changes in the macOS implementation of MGLMapView. Apart from annotation views (#5035), we’re trying to keep the annotation code consistent between the two platforms.

@RomainQuidet
Copy link
Contributor Author

Ok I'll do it tomorrow for MacOS.

@RomainQuidet
Copy link
Contributor Author

RomainQuidet commented Oct 20, 2016

Fun to see that method (MGLAnnotationTag)annotationTagAtPoint:(NSPoint)point persistingResults:(BOOL)persist from MGLMapView.mm in macos target has already the correct queryRect size I patched in iOS, meaning the way I patched iOS is correct! As macos does not handle yet MGLAnnotationViews, there's no need to update macos code. The click detection works well on this platform.

@1ec5
Copy link
Contributor

1ec5 commented Oct 20, 2016

Ah, OK. I may be mistaken, but the reason may be that clicking on macOS is much more precise than tapping on iOS, so there was never a need to introduce a “slop area” around an annotation.

@margvl
Copy link

margvl commented Aug 17, 2017

Hello, in our iOS application we are migrating from MKMapKit to Mapbox, however we are facing an issue, which may be related to this branch. Our annotations selection area is to big and user often selects a different annotations, we did not have this problem in MKMapKit.
Just wanted to know the status of this branch, because it was created in 2016.
Can I expect that it will be merged in near future?

@stale
Copy link

stale bot commented Oct 25, 2018

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the archived Archived because of inactivity label Oct 25, 2018
@stale
Copy link

stale bot commented Oct 25, 2018

This pull request has been automatically detected as stale because it has not had recent activity and will be archived. Thank you for your contributions.

@stale stale bot closed this Oct 25, 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 archived Archived because of inactivity iOS Mapbox Maps SDK for iOS
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants