Remove unnecessary spread operator

[dkniffin]

Fixes #11

[dkniffin]

@osvaldasvalutis Would you mind taking a look at this and merging it? Thanks!

[osvaldasvalutis]

@dkniffin the spread operator here converts NodeList to Array which allows to run forEach on older browsers (e.g. IE).

[dkniffin]

@osvaldasvalutis Ah, ok. I'll switch to Array.from. Will that be okay?

[osvaldasvalutis]

@dkniffin what's the reasoning behind that?

[dkniffin]

@osvaldasvalutis I described the issue over on #11, but in summary, the spread operator combined with the querySelectorAll intermittently has issues on some browsers (the ones we're seeing reports for are Chrome 84 & 85 on Windows, but it's hard to reproduce). I would expect Babel to resolve any issue like that, but it seems it doesn't. There's a similar issue here: babel/babel#10509

[osvaldasvalutis]

@dkniffin 👌 https://github.com/kollegorna/cocoon-vanilla-js/releases/tag/v1.0.7