1. Add callbacks; 2. Move private methods to the begin of methods list; 3. All...

1. Add callbacks; 2. Move private methods to the begin of methods list; 3. All JS types starts with Uppercase letter

shower.js

@@ -17,8 +17,8 @@ window.shower = (function(window, document, undefined) {
 	* Get value at named data store for the DOM element.
 	* @private
 	* @param {domElem} element
-	* @param {string} name
-	* @returns {string}
+	* @param {String} name
+	* @returns {String}
 	*/
 	shower._getData = function(element, name) {
 		return element.dataset ? element.dataset[name] : element.getAttribute('data-' + name);
@@ -55,7 +55,7 @@ window.shower = (function(window, document, undefined) {
 	/**
 	* Set CSS transform with prefixes to body
 	* @private
-	* @returns {true}
+	* @returns {Boolean}
 	*/
 	shower._applyTransform = function(transform) {
 		body.style.WebkitTransform = transform;
@@ -67,12 +67,104 @@ window.shower = (function(window, document, undefined) {
 		return true;
 	};
 
+	/**
+	* Check if arg is number.
+	* @private
+	* @param {Number|whatelse} arg Any type
+	* @returns {Boolean}
+	*/
+	shower._isNumber = function(arg) {
+		if ( ! ( ! isNaN(parseFloat(arg)) && isFinite(arg))) {
+			return false;
+		}
+
+		return true;
+	};
+
+	/**
+	* Normalize slide number.
+	* @private
+	* @param {Number} slideNumber slide number (sic!)
+	* @returns {Number}
+	*/
+	shower._normalizeSlideNumber = function(slideNumber) {
+		if ( ! shower._isNumber(slideNumber)) {
+			throw new Error('Gimme slide number as Number, baby!');
+		}
+
+		if (slideNumber < 0) {
+			slideNumber = 0;
+		}
+
+		if (slideNumber >= slideList.length) {
+			slideNumber = slideList.length - 1;
+		}
+
+		return slideNumber;
+	};
+
+	/**
+	* Get containing slide id.
+	* @private
+	* @param {domElem} el
+	* @returns {String}
+	*/
+	shower._getContainingSlideId = function(el) {
+		while ('BODY' !== el.nodeName && 'HTML' !== el.nodeName) {
+			if (el.classList.contains('slide')) {
+				return el.id;
+			} else {
+				el = el.parentNode;
+			}
+		}
+
+		return '';
+	};
+
+	/**
+	* Dispatch single slide mode.
+	* @TODO: Renaming needed? Or just some handlers rewriting?
+	* @private
+	* @param {domElem} e
+	* @returns {Undefined}
+	*/
+	shower._dispatchSingleSlideMode = function(e) {
+		// Process links
+		// @TODO: presentation links support
+		if ('A' === e.target.nodeName) {
+			e.preventDefault();
+
+			window.open(e.target.getAttribute('href'));
+			return;
+		}
+
+		var slideId = shower._getContainingSlideId(e.target);
+
+		if ('' !== slideId && shower.isListMode()) {
+			e.preventDefault();
+
+			// NOTE: we should update hash to get things work properly
+			url.hash = '#' + slideId;
+			if (isHistoryApiSupported) {
+				history.replaceState(null, null, url.pathname + '?full#' + slideId);
+			}
+			shower.enterSlideMode();
+
+			shower.updateProgress(shower.getCurrentSlideNumber());
+			shower.updateCurrentAndPassedSlides(shower.getCurrentSlideNumber());
+			shower.runSlideshowIfPresented(shower.getCurrentSlideNumber());
+		}
+
+		return;
+	};
+
 	/**
 	* Show next slide. If slide is last returns false, otherwise return slide
 	* number which been shown.
-	* @returns {number|false}
+	* @param {Function} callback runs only if shower.next() complete successfully
+	* @returns {Number|Boolean}
 	*/
-	shower.next = function() {
+	shower.next = function(callback) {
 		var currentSlideNumber = shower.getCurrentSlideNumber(),
 			ret;
 
@@ -87,6 +179,10 @@ window.shower = (function(window, document, undefined) {
 			shower.go(currentSlideNumber + 1);
 			// slides starts from 0
 			ret = currentSlideNumber + 2;
+
+			if (typeof(callback) === "function") {
+				callback();
+			}
 		} else {
 			ret = false;
 		}
@@ -97,9 +193,10 @@ window.shower = (function(window, document, undefined) {
 	/**
 	* Show previous slide. If slide is first returns false, otherwise return slide
 	* number which been shown.
-	* @returns {number|false}
+	* @param {Function} callback runs only if shower.previous() complete successfully
+	* @returns {Number|Boolean}
 	*/
-	shower.previous = function() {
+	shower.previous = function(callback) {
 		var currentSlideNumber = shower.getCurrentSlideNumber(),
 			ret;
 
@@ -107,6 +204,10 @@ window.shower = (function(window, document, undefined) {
 		if (currentSlideNumber > 0) {
 			ret = currentSlideNumber;
 			shower.go(currentSlideNumber - 1);
+
+			if (typeof(callback) === "function") {
+				callback();
+			}
 		} else {
 			ret = false;
 		}
@@ -116,47 +217,74 @@ window.shower = (function(window, document, undefined) {
 
 	/**
 	* Show first slide.
-	* @returns {number}
+	* @param {Function} callback
+	* @returns {Number}
 	*/
-	shower.first = function() {
+	shower.first = function(callback) {
+		if (typeof(callback) === "function") {
+			callback();
+		}
+
 		return shower.go(0);
 	};
 
 	/**
 	* Show last slide.
-	* @returns {number}
+	* @param {Function} callback
+	* @returns {Number}
 	*/
-	shower.last = function() {
+	shower.last = function(callback) {
+		if (typeof(callback) === "function") {
+			callback();
+		}
 		return shower.go(slideList.length - 1);
 	};
 
 	/**
 	* Switch to slide view.
-	* @returns {number}
+	* @param {Function} callback runs only if shower.enterSlideMode() complete successfully
+	* @returns {Number|Boolean}
 	*/
-	shower.enterSlideMode = function() {
+	shower.enterSlideMode = function(callback) {
 		// check if it's already in slide mode...
-		if (body.classList.contains('full')) { return; }
+		if (body.classList.contains('full')) {
+			return false;
+		}
 
 		body.classList.remove('list');
 		body.classList.add('full');
 
-		if (window.console && window.console.clear) { console.clear(); }
+		if (window.console && window.console.clear) {
+			console.clear();
+		}
+
 		shower.showPresenterNotes(shower.getCurrentSlideNumber());
 
+		if (typeof(callback) === "function") {
+			callback();
+		}
+
 		return shower._applyTransform(shower._getTransform());
 	};
 
 	/**
 	* Switch to list view.
-	* @returns {number}
+	* @param {Function} callback runs only if shower.enterListMode() complete successfully
+	* @returns {Number}
 	*/
-	shower.enterListMode = function() {
+	shower.enterListMode = function(callback) {
 		// check if it's already in list mode...
-		if (body.classList.contains('list')) { return; }
+		if (body.classList.contains('list')) {
+			return false;
+		}
 
 		body.classList.remove('full');
 		body.classList.add('list');
+
+		if (typeof(callback) === "function") {
+			callback();
+		}
+
 		return shower._applyTransform('none');
 	};
 
@@ -166,7 +294,7 @@ window.shower = (function(window, document, undefined) {
 	* Get current slide number. Starts from zero. Warning: when in url you have
 	* slide number 1 this method will return 0.
 	* If something wrong return -1.
-	* @returns {number}
+	* @returns {Number}
 	*/
 	shower.getCurrentSlideNumber = function() {
 		var i = slideList.length - 1,
@@ -183,35 +311,21 @@ window.shower = (function(window, document, undefined) {
 		return -1;
 	};
 
-	/**
-	* Check if arg is number.
-	* @private
-	* @param {number|whatelse} arg Any type
-	* @returns {boolean}
-	*/
-	shower._isNumber = function(arg) {
-		if ( ! ( ! isNaN(parseFloat(arg)) && isFinite(arg))) {
-			return false;
-		}
-
-		return true;
-	};
-
 	/**
 	* Scroll to slide.
-	* @param {number} slideNumber slide number (sic!)
-	* @returns {undefined|boolean}
+	* @param {Number} slideNumber slide number (sic!)
+	* @returns {Undefined|Boolean}
 	*/
 	shower.scrollToSlide = function(slideNumber) {
 		var currentSlide,
 			ret;
 
 		if ( ! shower._isNumber(slideNumber)) {
-			throw new Error('Gimme slide number as number, baby!');
+			throw new Error('Gimme slide number as Number, baby!');
 		}
 
 		if ( ! shower.isListMode()) {
-			throw new Error('You can\'t to scroll cause you in slide mode. Please, switch to list mode.');
+			throw new Error('You can\'t scroll to because you in slide mode. Please, switch to list mode.');
 		}
 
 		// @TODO: WTF?
@@ -232,38 +346,16 @@ window.shower = (function(window, document, undefined) {
 
 	/**
 	* Chech if it's list mode.
-	* @returns {boolean}
+	* @returns {Boolean}
 	*/
 	shower.isListMode = function() {
 		return isHistoryApiSupported ? ! /^full.*/.test(url.search.substr(1)) : body.classList.contains('list');
 	};
 
-	/**
-	* Normalize slide number.
-	* @private
-	* @param {number} slideNumber slide number (sic!)
-	* @returns {number}
-	*/
-	shower._normalizeSlideNumber = function(slideNumber) {
-		if ( ! shower._isNumber(slideNumber)) {
-			throw new Error('Gimme slide number as number, baby!');
-		}
-
-		if (slideNumber < 0) {
-			slideNumber = 0;
-		}
-
-		if (slideNumber >= slideList.length) {
-			slideNumber = slideList.length - 1;
-		}
-
-		return slideNumber;
-	};
-
 	/**
 	* Update progress bar.
-	* @param {number} slideNumber slide number (sic!)
-	* @returns {boolean}
+	* @param {Number} slideNumber slide number (sic!)
+	* @returns {Boolean}
 	*/
 	shower.updateProgress = function(slideNumber) {
 		// if progress bar doesn't exist
@@ -272,7 +364,7 @@ window.shower = (function(window, document, undefined) {
 		}
 
 		if ( ! shower._isNumber(slideNumber)) {
-			throw new Error('Gimme slide number as number, baby!');
+			throw new Error('Gimme slide number as Number, baby!');
 		}
 
 		progress.style.width = (100 / (slideList.length - 1) * shower._normalizeSlideNumber(slideNumber)).toFixed(2) + '%';
@@ -282,8 +374,8 @@ window.shower = (function(window, document, undefined) {
 
 	/**
 	* Update current and passed slides.
-	* @param {number} slideNumber slide number (sic!)
-	* @returns {boolean}
+	* @param {Number} slideNumber slide number (sic!)
+	* @returns {Boolean}
 	*/
 	shower.updateCurrentAndPassedSlides = function(slideNumber) {
 		var i,
@@ -293,7 +385,7 @@ window.shower = (function(window, document, undefined) {
 		slideNumber = shower._normalizeSlideNumber(slideNumber);
 
 		if ( ! shower._isNumber(slideNumber)) {
-			throw new Error('Gimme slide number as number, baby!');
+			throw new Error('Gimme slide number as Number, baby!');
 		}
 
 		for (i = 0; i < l; ++i) {
@@ -316,7 +408,7 @@ window.shower = (function(window, document, undefined) {
 
 	/**
 	* Show presenter notes in console.
-	* @param {number} slideNumber slide number (sic!). Attention: starts from zero.
+	* @param {Number} slideNumber slide number (sic!). Attention: starts from zero.
 	*/
 	shower.showPresenterNotes = function(slideNumber) {
 		if (window.console) {
@@ -341,12 +433,12 @@ window.shower = (function(window, document, undefined) {
 
 	/**
 	* Get slide hash.
-	* @param {number} slideNumber slide number (sic!). Attention: starts from zero.
-	* @returns {boolean}
+	* @param {Number} slideNumber slide number (sic!). Attention: starts from zero.
+	* @returns {Boolean}
 	*/
 	shower.getSlideHash = function(slideNumber) {
 		if ( ! shower._isNumber(slideNumber)) {
-			throw new Error('Gimme slide number as number, baby!');
+			throw new Error('Gimme slide number as Number, baby!');
 		}
 
 		slideNumber = shower._normalizeSlideNumber(slideNumber);
@@ -356,12 +448,13 @@ window.shower = (function(window, document, undefined) {
 
 	/**
 	* Go to slide number...
-	* @param {number} slideNumber slide number (sic!). Attention: starts from zero.
-	* @returns {number}
+	* @param {Number} slideNumber slide number (sic!). Attention: starts from zero.
+	* @param {Function} callback runs only if you not in List mode
+	* @returns {Number}
 	*/
-	shower.go = function(slideNumber) {
+	shower.go = function(slideNumber, callback) {
 		if ( ! shower._isNumber(slideNumber)) {
-			throw new Error('Gimme slide number as number, baby!');
+			throw new Error('Gimme slide number as Number, baby!');
 		}
 
 		url.hash = shower.getSlideHash(slideNumber);
@@ -370,74 +463,23 @@ window.shower = (function(window, document, undefined) {
 			shower.updateProgress(slideNumber);
 			shower.updateCurrentAndPassedSlides(slideNumber);
 			shower.showPresenterNotes(slideNumber);
-		}
-
-		return slideNumber;
-	};
-
-	/**
-	* Get containing slide id.
-	* @private
-	* @param {domElem} el
-	* @returns {string}
-	*/
-	shower._getContainingSlideId = function(el) {
-		while ('BODY' !== el.nodeName && 'HTML' !== el.nodeName) {
-			if (el.classList.contains('slide')) {
-				return el.id;
-			} else {
-				el = el.parentNode;
-			}
-		}
 
-		return '';
-	};
-
-	/**
-	* Dispatch single slide mode.
-	* @TODO: Renaming needed? Or just some handlers rewriting?
-	* @private
-	* @param {domElem} e
-	* @returns {undefined}
-	*/
-	shower._dispatchSingleSlideMode = function(e) {
-		// Process links
-		// @TODO: presentation links support
-		if ('A' === e.target.nodeName) {
-			e.preventDefault();
-
-			window.open(e.target.getAttribute('href'));
-			return;
-		}
-
-		var slideId = shower._getContainingSlideId(e.target);
-
-		if ('' !== slideId && shower.isListMode()) {
-			e.preventDefault();
-
-			// NOTE: we should update hash to get things work properly
-			url.hash = '#' + slideId;
-			if (isHistoryApiSupported) {
-				history.replaceState(null, null, url.pathname + '?full#' + slideId);
+			if (typeof(callback) === "function") {
+				callback();
 			}
-			shower.enterSlideMode();
-
-			shower.updateProgress(shower.getCurrentSlideNumber());
-			shower.updateCurrentAndPassedSlides(shower.getCurrentSlideNumber());
-			shower.runSlideshowIfPresented(shower.getCurrentSlideNumber());
 		}
 
-		return;
+		return slideNumber;
 	};
 
 	/**
-	* Run slide show in presented.
-	* @param {number} slideNumber
-	* @returns {undefined}
+	* Run slide show if presented.
+	* @param {Number} slideNumber
+	* @returns {Undefined}
 	*/
 	shower.runSlideshowIfPresented = function(slideNumber) {
 		if ( ! shower._isNumber(slideNumber)) {
-			throw new Error('Gimme slide number as number, baby!');
+			throw new Error('Gimme slide number as Number, baby!');
 		}
 
 		slideNumber = shower._normalizeSlideNumber(slideNumber);
@@ -460,8 +502,8 @@ window.shower = (function(window, document, undefined) {
 
 	/**
 	* Increases inner navigation by adding 'active' class to next inactive inner navigation item
-	* @param {number} slideNumber
-	* @returns {number}
+	* @param {Number} slideNumber
+	* @returns {Number}
 	*/
 	shower.increaseInnerNavigation = function(slideNumber) {
 		var nextNodes,
@@ -469,7 +511,7 @@ window.shower = (function(window, document, undefined) {
 			ret = -1;
 
 		if ( ! shower._isNumber(slideNumber)) {
-			throw new Error('Gimme slide number as number, baby!');
+			throw new Error('Gimme slide number as Number, baby!');
 		}
 
 		// If inner navigation in this slide...