Cloning objects (Object spread)

[akinsho]

Hi,

Hope learning react is going okay, having a look through the code and I noticed that here you are cloning an object by JSON.stringifying then parsing a simpler and arguably more readably solution would be to use Object spread notation ... (not sure if your current build strategy supports it) example: let objClone = { ...obj };

[oliverjam]

We didn't set up Babel for Object spread support, but you could use Object.assign to achieve the same result.

Actually since cells is an array you can use array spread to copy it ([...array])

[isnotafunction]

Thanks so much for these suggestions! I thought spread does shallow copying, so just curious if that would ever be a problem?

[oliverjam]

Yeah it does do shallow copying, but in this case that would be fine. If you have nested objects/arrays you would need to do something a bit more complex

[isnotafunction]

Thanks! 👍