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

Switch to DOM methods to avoid XSS attack #5

Closed healiha closed 9 years ago

healiha commented 9 years ago

As suggested by @Daniel15 suggested on #4 i replaced 

var empty_row = '<' + opts.tag + ' class="' + opts.no_data_class + '">';

by DOM methods like document.createElement.

NeXTs commented 9 years ago

Hi! Thank you for your contribution I accepted your pull request, but refactored it a little bit. Hope you don't mind :)

healiha commented 9 years ago

Of course not, I'm glad I could help! :)