Serve non-minified version of jump.js about jump.js HOT 6 CLOSED

I’m just taking a wild guess the "main": "dist/jump.min.js" in the package.json is what’s npm is using to decide which file to use? I vote +1 to change it to the non-minified file.

from jump.js.

Correct, that's what npm is using to decide on the minified file. I think this requires a small code change to change the build script to not minify, but I'm not 100% sure.

from jump.js.

@vincentorback @saarons makes sense, since people using this from NPM must be using a module bundler and are likely minifying their build anyways. I want to support ES6 practices and workflow above all else, and I think this suggestion is in line with that.

the relevant line of the gulpfile is here. removing that will keep the dist file from being minified. if it's not minified, it would also be wise to rename it, removing the .min.

this weekend, i'm planning on un-minifying all my dist files, for all my libraries, as well as adding the jsnext:main field to all my package.jsons, to support future-facing tree-shaking module bundlers like rollup

from jump.js.

Will this be implemented any time soon?

from jump.js.

For those waiting, Sitepoint offers a ES5 version in an article about this lib. Don't know the diff to the last version, but seems to work and includes an accessibility bugfix as a bonus. http://www.sitepoint.com/smooth-scrolling-vanilla-javascript/

(Note: I'm not affiliated with them, just thought it is worth pointing out while this case is open)

from jump.js.

please see #31, and leave your feedback!

from jump.js.