-
Notifications
You must be signed in to change notification settings - Fork 50
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
Fixed Annotations HitTesting & Added annotation anchor #35
Conversation
Fixing annotation centering
Thank you for your contribution. I will open a similar pull request in the coming days, since I disagree with the addition of directions into this library (you could easily add this right now, so it doesn't need to be integrated in here). Also it seems that you dropped watchOS 6.0 support - however, this library tries to keep supporting all platforms starting at watchOS 6, macOS 10.15, iOS 13.0 and tvOS 13.0. |
@pauljohanneskraft I would be happy to open new PR with these changes if you don't mind. |
That sounds really great! Yes, thank you 😊 |
This PR implements several fixes and enchantments:
anchor
parameter to theViewMapAnnotation
similar to the default SwiftUI Map approach.P.S.

@pauljohanneskraft
The problem with your hittesting solution in the hit-testing branch was that the SwiftUI annotation view was not constrained to the underlying UIView. Therefore, when you moved the map, the SwiftUI view's position did not change. You may see in the picture below.