Skip to content

Conversation

aramsargsyan
Copy link

This is an option for users of the library to implement animations in a custom way. Giving out the [CKClusterAnimation] array gives the possibility to implement the animations in any desirable way. I added another method in the delegate instead of changing the current one for the existing users of the library not to have to change much code.

@maxep
Copy link
Contributor

maxep commented Jul 17, 2020

Nice work here @aramsargsyan and sorry for the late reply.

Instead of adding the type as a CKClusterAnimation property, the enum value could be pass as an argument of the MKMap method performAnimations:completion:. Then the additional delegate method signature would look like:

- (void)clusterManager:(CKClusterManager *)clusterManager performAnimations:(void (^)(void))animations type:(CKClusterAnimationType)type completion:(void (^ __nullable)(BOOL finished))completion;

In order to be merged, you will also need to update MapKit, GoogleMaps and YandexMapKit extensions.

Cluster coordinate.
*/
@property (nonatomic, readonly) CLLocationCoordinate2D coordinate;
@property (nonatomic) CLLocationCoordinate2D coordinate;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't think this change is required as CKCluster implements the MKAnnotation method setCoordinate:.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants