search

MoOx / pjax

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

Minify pjax.js #108

Closed BehindTheMath closed 6 years ago

BehindTheMath commented 6 years ago

We should add a minified version of pjax.js to the NPM package, to reduce the bandwith required for anyone linking to it directly (using unpkg.com).

jimaek commented 6 years ago

jsDelivr can auto-minify css and js files. Check it out http://cdn.jsdelivr.net/npm/pjax@0.2.4/pjax.min.js

BehindTheMath commented 6 years ago

Cool. Maybe we should suggest this in the README for the unminified version as well, instead of Unpkg.

MoOx commented 6 years ago

Can be added to npm package, but imo should not be versionned (can quickly become a pain to maintain). You can exclude with gitignore but include with npm package.json/files section.

BehindTheMath commented 6 years ago

but imo should not be versionned

Do you mean it shouldn't be committed to Github?

MoOx commented 6 years ago

Exactly.