Skip to content

Add deep clone for cluster mapping#216

Open
mrkwanzaa wants to merge 9 commits intomapbox:mainfrom
mrkwanzaa:deep-clone
Open

Add deep clone for cluster mapping#216
mrkwanzaa wants to merge 9 commits intomapbox:mainfrom
mrkwanzaa:deep-clone

Conversation

@mrkwanzaa
Copy link

This closes #215 by adding a simple deep clone algorithm and using that in _map

@oscarfonts
Copy link

I just came across this same issue, and the patch proposed in this PR fixes it.

It doesn't have a performance impact (according to npm run bench numbers), and it has no dependencies.

How can I help getting this merged?

Thanks.

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.

Nested geoJson properties are copied by reference

2 participants