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

Add a container view for UIView based annotations #5140

Closed
boundsj opened this issue May 25, 2016 · 1 comment
Closed

Add a container view for UIView based annotations #5140

boundsj opened this issue May 25, 2016 · 1 comment
Assignees
Labels
annotations Annotations on iOS and macOS or markers on Android iOS Mapbox Maps SDK for iOS performance Speed, stability, CPU usage, memory usage, or power usage refactor
Milestone

Comments

@boundsj
Copy link
Contributor

boundsj commented May 25, 2016

In #4801 we added the ability for application code to provide UIView based representations for annotations. Currently, the annotation views are added as subviews of the map view. This has potential performance implications and is probably the source if issues like this.

We should add a container view similar to MapKit to contain the annotation views and perhaps other containers for other UIView based visuals. This should help reduce the complexity of layering UIView based visuals over the map.

MapKit annotation container

screen shot 2016-05-25 at 12 09 37 pm

@boundsj boundsj added iOS Mapbox Maps SDK for iOS performance Speed, stability, CPU usage, memory usage, or power usage refactor annotations Annotations on iOS and macOS or markers on Android labels May 25, 2016
@boundsj boundsj added this to the ios-v3.3.0 milestone May 25, 2016
@boundsj boundsj self-assigned this May 25, 2016
@boundsj
Copy link
Contributor Author

boundsj commented May 26, 2016

@1ec5 @friedbunny I created WIP PR (#5148) for your review.

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 performance Speed, stability, CPU usage, memory usage, or power usage refactor
Projects
None yet
Development

No branches or pull requests

1 participant