search

MoOx / pjax

Easily enable fast Ajax navigation on any website (using pushState + xhr)
MIT License
1.46k stars 125 forks source link

Replace instances of ES6 `const` keyword with `var` #107

Closed robinnorth closed 6 years ago

robinnorth commented 6 years ago

Fixes #106

BehindTheMath commented 6 years ago

Thanks.

MoOx commented 6 years ago

IMO we should keep es6 syntax and use babel to transpile for old browser when creating a bundle.

BehindTheMath commented 6 years ago

We can do that, but it will mean a major revamp of the code, which will include removing a bunch of legacy code, like the polyfills for Function.prototype.bind, and Array.prototype.forEach.

I think that should wait until we finish fixing all the existing issues, so we don't cause any conflicts. I'll open a new issue to track that.

MoOx commented 6 years ago

Make sense