Expose API via CommonJS instead of a global variable, when supported (#…

…38) * Expose API via CommonJS instead of a global variable, when supported * use double quotes :( * bump verison number, this is a breaking change
LeaVerou · Jan 8, 2019 · 2be1637 · 2be1637
1 parent 7373709
commit 2be1637
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 7 deletions.
diff --git a/multirange.js b/multirange.js
@@ -5,7 +5,7 @@ var supportsMultiple = self.HTMLInputElement && "valueLow" in HTMLInputElement.p
 
 var descriptor = Object.getOwnPropertyDescriptor(HTMLInputElement.prototype, "value");
 
-self.multirange = function(input) {
+var multirange = function(input) {
 	if (supportsMultiple || input.classList.contains("multirange")) {
 		return;
 	}
@@ -76,11 +76,16 @@ multirange.init = function() {
 	[].slice.call(document.querySelectorAll("input[type=range][multiple]:not(.multirange)")).forEach(multirange);
 }
 
-if (document.readyState == "loading") {
-	document.addEventListener("DOMContentLoaded", multirange.init);
-}
-else {
-	multirange.init();
+if (typeof module === "undefined") {
+	self.multirange = multirange;
+	if (document.readyState == "loading") {
+		document.addEventListener("DOMContentLoaded", multirange.init);
+	}
+	else {
+		multirange.init();
+	}
+} else {
+	module.exports = multirange;
 }
 
 })();
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "multirange",
-  "version": "1.0.0",
+  "version": "2.0.0",
   "description": "A tiny polyfill for HTML5 multi-handle sliders",
   "main": "multirange.js",
   "scripts": {