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

creating global variable 'is_mac' #144

Closed sgratzl closed 6 years ago

sgratzl commented 6 years ago

as far as I can see:

https://github.com/NeXTs/Clusterize.js/blob/7cd274471e050621a07be3cb40b4fd32e69812a3/clusterize.js#L23

creates a global variable is_mac since the var keyword is missing

NeXTs commented 6 years ago

Hello Samuel

Thanks, but it doesn't create global variable because of var declared higher, for ie and , between them.

sgratzl commented 6 years ago

ups I missed that comma