-
Notifications
You must be signed in to change notification settings - Fork 697
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
deselectAnnotation and annotationsPopUpEnabled #418
Conversation
BOOL _attributionButton; | ||
BOOL _logo; | ||
BOOL _compass; | ||
UIEdgeInsets _contentInset; | ||
MGLAnnotationVerticalAlignment _userLocationVerticalAlignment; | ||
|
||
BOOL _annotationsPopUpEnabled; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Make this a property so you don't need to implement the setter yourself.
_selectedAnnotation = nil; | ||
return; | ||
} | ||
_selectedAnnotation = annotation; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
MGLMapView has a selectedAnnotations property for this.
Tested this is working on iOS and Android. Merging. |
Thank you so much, @dapetcu21. Any idea when this package will get updated in the npm registry? |
@bsudekum should have access to npm. Meanwhile you can |
…ltilesource Fix missing rctmgltilesource
Rebase of #386. Will test tomorrow to see if everything's ok, then merge.