Fix/update cluster changes

[marcjulian]

Fixes #188

[HarelM]

I've noticed that the cluster code keeps refreshing even if nothing changes (I've added a console.log in updateCluster method in MarkersForClustersComponent) do you think it's the right behavior?

[marcjulian]

I've noticed that the cluster code keeps refreshing even if nothing changes (I've added a console.log in updateCluster method in MarkersForClustersComponent) do you think it's the right behavior?

I added the same log in updateCluster before #178 was merged. The function is only triggered once at the beginning and when zoomed. I guess its not right that it keeps calling this function after #178 without any interaction.

[HarelM]

Yup...

[HarelM]

I wonder if there are other places where this happens.
Cc: @n-elhk.

[marcjulian]

@HarelM I'm using now afterRender to call applyChanges instead of zone.onMicrotaskEmpty. I also renamed applyChanges to clearMapElements, what do you think about the name?

I think the case where updateCluster is being called multiple times is unrelated with this issue, perhaps we should open a new issue and pr for that.

[HarelM]

Looks good to me.

[HarelM]

Feel free to open a different issue, whatever work for you, I'm good with it.
I would simply like to make sure we solve this as this is a performance issue from my point of view.