search

NeXTs / Clusterize.js

Tiny vanilla JS plugin to display large data sets easily
https://clusterize.js.org
MIT License
7.22k stars 412 forks source link

Replace loop by converting node list to array #90

Closed thomas-rusz closed 7 years ago

thomas-rusz commented 7 years ago

What do you think about slice instead of iterating through all the elements and pushing them into a new array?

NeXTs commented 7 years ago

Hey

Thanks, trick with [].slice.call was used at beginning but later was changed to cycle in this commit

[].slice.call won't work at ie8

thomas-rusz commented 7 years ago

Oh, I see. Thank you for your quick answer!