-
Notifications
You must be signed in to change notification settings - Fork 131
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
[viewannotation] Introduce show annotations api [MAPSAND-464] #1753
Conversation
48634cd
to
4ca6825
Compare
sdk/src/main/java/com/mapbox/maps/viewannotation/ViewAnnotationManager.kt
Outdated
Show resolved
Hide resolved
sdk/src/main/java/com/mapbox/maps/viewannotation/ViewAnnotationManager.kt
Outdated
Show resolved
Hide resolved
b21127f
to
a3a0135
Compare
29d26ea
to
6155c4a
Compare
6155c4a
to
555087a
Compare
sdk/src/main/java/com/mapbox/maps/viewannotation/ViewAnnotationManager.kt
Outdated
Show resolved
Hide resolved
sdk/src/main/java/com/mapbox/maps/viewannotation/ViewAnnotationOptionsExt.kt
Outdated
Show resolved
Hide resolved
sdk/src/main/java/com/mapbox/maps/viewannotation/ViewAnnotationOptionsExt.kt
Outdated
Show resolved
Hide resolved
sdk/src/main/java/com/mapbox/maps/viewannotation/ViewAnnotationOptionsExt.kt
Outdated
Show resolved
Hide resolved
sdk/src/main/java/com/mapbox/maps/viewannotation/ViewAnnotationOptionsExt.kt
Outdated
Show resolved
Hide resolved
8b5cbd9
to
7a60489
Compare
sdk/src/main/java/com/mapbox/maps/viewannotation/ViewAnnotationOptionsExt.kt
Outdated
Show resolved
Hide resolved
sdk/src/main/java/com/mapbox/maps/viewannotation/ViewAnnotationOptionsExt.kt
Outdated
Show resolved
Hide resolved
ce27415
to
c121b28
Compare
sdk/src/main/java/com/mapbox/maps/viewannotation/ViewAnnotationManager.kt
Show resolved
Hide resolved
eb30475
to
da33a32
Compare
b78ca74
to
58854ac
Compare
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.
LGTM, one nit
sdk/src/main/java/com/mapbox/maps/viewannotation/ViewAnnotationManager.kt
Outdated
Show resolved
Hide resolved
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.
Couple questions
sdk/src/main/java/com/mapbox/maps/viewannotation/ViewAnnotationManager.kt
Show resolved
Hide resolved
sdk/src/main/java/com/mapbox/maps/viewannotation/ViewAnnotationManager.kt
Show resolved
Hide resolved
58854ac
to
9692f51
Compare
sdk/src/main/java/com/mapbox/maps/viewannotation/ViewAnnotationManager.kt
Show resolved
Hide resolved
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.
LGTM with one question
Summary of changes
Introduce
ViewAnnotationManager.cameraForAnnotations
that returnsCameraOptions
for provided annotations. Users can use custom animation to move camera to fit specified anntations.If users set
ViewAnnotationOptions.visible = false
explicitly, we will exclude those views from the calculation forcameraOptions
. Also, only the views for which visibility is set to Visible, will be considered in the calculation.User impact (optional)
viewannotation_framing.mp4