expose layout api method (closes #305), consistent spacing across all functions in core JS, remove unused each() method

js/reveal.js

@@ -88,6 +88,7 @@ var Reveal = (function(){
 								'OPerspective' in document.body.style ||
 								'perspective' in document.body.style,
 
+		// Detect support for CSS 2D transforms
 		supports2DTransforms =  'WebkitTransform' in document.body.style ||
 								'MozTransform' in document.body.style ||
 								'msTransform' in document.body.style ||
@@ -123,6 +124,7 @@ var Reveal = (function(){
 	 * Starts up the presentation if the client is capable.
 	 */
 	function initialize( options ) {
+
 		if( ( !supports2DTransforms && !supports3DTransforms ) ) {
 			document.body.setAttribute( 'class', 'no-transforms' );
 
@@ -151,6 +153,7 @@ var Reveal = (function(){
 	 * not found, it is created.
 	 */
 	function setupDOM() {
+
 		// Cache references to key DOM elements
 		dom.theme = document.querySelector( '#theme' );
 		dom.wrapper = document.querySelector( '.reveal' );
@@ -204,12 +207,14 @@ var Reveal = (function(){
 			dom.controlsPrev = toArray( document.querySelectorAll( '.navigate-prev' ) );
 			dom.controlsNext = toArray( document.querySelectorAll( '.navigate-next' ) );
 		}
+
 	}
 
 	/**
 	 * Hides the address bar if we're on a mobile device.
 	 */
 	function hideAddressBar() {
+
 		if( navigator.userAgent.match( /(iphone|ipod)/i ) ) {
 			// Give the page some scrollable overflow
 			document.documentElement.style.overflow = 'scroll';
@@ -219,6 +224,7 @@ var Reveal = (function(){
 			window.addEventListener( 'load', removeAddressBar, false );
 			window.addEventListener( 'orientationchange', removeAddressBar, false );
 		}
+
 	}
 
 	/**
@@ -229,6 +235,7 @@ var Reveal = (function(){
 	 * will load after reveal.js has been started up.
 	 */
 	function load() {
+
 		var scripts = [],
 			scriptsAsync = [];
 
@@ -270,6 +277,7 @@ var Reveal = (function(){
 		else {
 			proceed();
 		}
+
 	}
 
 	/**
@@ -277,6 +285,7 @@ var Reveal = (function(){
 	 * to the current URL deeplink if there is one.
 	 */
 	function start() {
+
 		// Make sure we've got all the DOM elements we need
 		setupDOM();
 
@@ -305,12 +314,14 @@ var Reveal = (function(){
 				'currentSlide': currentSlide
 			} );
 		}, 1 );
+
 	}
 
 	/**
 	 * Applies the configuration settings from the config object.
 	 */
 	function configure() {
+
 		if( supports3DTransforms === false ) {
 			config.transition = 'linear';
 		}
@@ -356,12 +367,14 @@ var Reveal = (function(){
 				dom.theme.setAttribute( 'href', themeURL );
 			}
 		}
+
 	}
 
 	/**
 	 * Binds all event listeners.
 	 */
 	function addEventListeners() {
+
 		document.addEventListener( 'touchstart', onDocumentTouchStart, false );
 		document.addEventListener( 'touchmove', onDocumentTouchMove, false );
 		document.addEventListener( 'touchend', onDocumentTouchEnd, false );
@@ -385,12 +398,14 @@ var Reveal = (function(){
 			dom.controlsPrev.forEach( function( el ) { el.addEventListener( actionEvent, preventAndForward( navigatePrev ), false ); } );
 			dom.controlsNext.forEach( function( el ) { el.addEventListener( actionEvent, preventAndForward( navigateNext ), false ); } );
 		}
+
 	}
 
 	/**
 	 * Unbinds all event listeners.
 	 */
 	function removeEventListeners() {
+
 		document.removeEventListener( 'keydown', onDocumentKeyDown, false );
 		document.removeEventListener( 'touchstart', onDocumentTouchStart, false );
 		document.removeEventListener( 'touchmove', onDocumentTouchMove, false );
@@ -411,6 +426,7 @@ var Reveal = (function(){
 			dom.controlsPrev.forEach( function( el ) { el.removeEventListener( actionEvent, preventAndForward( navigatePrev ), false ); } );
 			dom.controlsNext.forEach( function( el ) { el.removeEventListener( actionEvent, preventAndForward( navigateNext ), false ); } );
 		}
+
 	}
 
 	/**
@@ -418,22 +434,20 @@ var Reveal = (function(){
 	 * If there's a conflict, object b takes precedence.
 	 */
 	function extend( a, b ) {
+
 		for( var i in b ) {
 			a[ i ] = b[ i ];
 		}
+
 	}
 
 	/**
 	 * Converts the target object to an array.
 	 */
 	function toArray( o ) {
-		return Array.prototype.slice.call( o );
-	}
 
-	function each( targets, method, args ) {
+		return Array.prototype.slice.call( o );
-		targets.forEach( function( el ) {
+
-			el[method].apply( el, args );
-		} );
 	}
 
 	/**
@@ -444,10 +458,12 @@ var Reveal = (function(){
 	 * @param {Object} b point with x/y properties
 	 */
 	function distanceBetween( a, b ) {
+
 		var dx = a.x - b.x,
 			dy = a.y - b.y;
 
 		return Math.sqrt( dx*dx + dy*dy );
+
 	}
 
 	/**
@@ -458,10 +474,12 @@ var Reveal = (function(){
 	 * after the wrapper has been executed
 	 */
 	function preventAndForward( delegate ) {
+
 		return function( event ) {
 			event.preventDefault();
 			delegate.call( null, event );
 		};
+
 	}
 
 	/**
@@ -469,9 +487,11 @@ var Reveal = (function(){
 	 * more vertical space ftw.
 	 */
 	function removeAddressBar() {
+
 		setTimeout( function() {
 			window.scrollTo( 0, 1 );
 		}, 0 );
+
 	}
 
 	/**
@@ -479,16 +499,19 @@ var Reveal = (function(){
 	 * reveal DOM element.
 	 */
 	function dispatchEvent( type, properties ) {
+
 		var event = document.createEvent( "HTMLEvents", 1, 2 );
 		event.initEvent( type, true, true );
 		extend( event, properties );
 		dom.wrapper.dispatchEvent( event );
+
 	}
 
 	/**
 	 * Wrap all links in 3D goodness.
 	 */
 	function linkify() {
+
 		if( supports3DTransforms && !( 'msPerspective' in document.body.style ) ) {
 			var nodes = document.querySelectorAll( SLIDES_SELECTOR + ' a:not(.image)' );
 
@@ -506,6 +529,7 @@ var Reveal = (function(){
 				}
 			}
 		}
+
 	}
 
 	/**
@@ -525,7 +549,7 @@ var Reveal = (function(){
 			for( var i = 0, len = slides.length; i < len; i++ ) {
 				var slide = slides[ i ];
 
-				// Don't bother update invisible slides
+				// Don't bother updating invisible slides
 				if( slide.style.display === 'none' ) {
 					continue;
 				}
@@ -553,9 +577,11 @@ var Reveal = (function(){
 	 * @param {int} v Index to memorize
 	 */
 	function setPreviousVerticalIndex( stack, v ) {
+
 		if( stack ) {
 			stack.setAttribute( 'data-previous-indexv', v || 0 );
 		}
+
 	}
 
 	/**
@@ -566,11 +592,13 @@ var Reveal = (function(){
 	 * @param {HTMLElement} stack The vertical stack element
 	 */
 	function getPreviousVerticalIndex( stack ) {
+
 		if( stack && stack.classList.contains( 'stack' ) ) {
 			return parseInt( stack.getAttribute( 'data-previous-indexv' ) || 0, 10 );
 		}
 
 		return 0;
+
 	}
 
 	/**
@@ -593,7 +621,7 @@ var Reveal = (function(){
 
 			// Not the pretties solution, but need to let the overview 
 			// class apply first so that slides are measured accurately 
-			// before we can positon them
+			// before we can position them
 			activateOverviewTimeout = setTimeout( function(){
 
 				var horizontalSlides = document.querySelectorAll( HORIZONTAL_SLIDES_SELECTOR );
@@ -704,12 +732,14 @@ var Reveal = (function(){
 	 * overview is open, false means it's closed.
 	 */
 	function toggleOverview( override ) {
+
 		if( typeof override === 'boolean' ) {
 			override ? activateOverview() : deactivateOverview();
 		}
 		else {
 			isOverviewActive() ? deactivateOverview() : activateOverview();
 		}
+
 	}
 
 	/**
@@ -719,7 +749,9 @@ var Reveal = (function(){
 	 * false otherwise
 	 */
 	function isOverviewActive() {
+
 		return dom.wrapper.classList.contains( 'overview' );
+
 	}
 
 	/**
@@ -729,6 +761,7 @@ var Reveal = (function(){
 	 * @see https://developer.mozilla.org/en-US/docs/DOM/Using_fullscreen_mode
 	 */
 	function enterFullscreen() {
+
 		var element = document.body;
 
 		// Check which implementation is available
@@ -740,6 +773,7 @@ var Reveal = (function(){
 		if( requestMethod ) {
 			requestMethod.apply( element );
 		}
+
 	}
 
 	/**
@@ -747,33 +781,41 @@ var Reveal = (function(){
 	 * black.
 	 */
 	function pause() {
+
 		dom.wrapper.classList.add( 'paused' );
+
 	}
 
 	/**
 	 * Exits from the paused mode.
 	 */
 	function resume() {
+
 		dom.wrapper.classList.remove( 'paused' );
+
 	}
 
 	/**
 	 * Toggles the paused mode on and off.
 	 */
 	function togglePause() {
+
 		if( isPaused() ) {
 			resume();
 		}
 		else {
 			pause();
 		}
+
 	}
 
 	/**
 	 * Checks if we are currently in the paused mode.
 	 */
 	function isPaused() {
+
 		return dom.wrapper.classList.contains( 'paused' );
+
 	}
 
 	/**
@@ -787,6 +829,7 @@ var Reveal = (function(){
 	 * target slide to activate
 	 */
 	function slide( h, v, f ) {
+
 		// Remember where we were at before
 		previousSlide = currentSlide;
 
@@ -837,7 +880,7 @@ var Reveal = (function(){
 			dispatchEvent( state[i] );
 		}
 
-		// Clean up the remaints of the previous state
+		// Clean up the remains of the previous state
 		while( stateBefore.length ) {
 			document.documentElement.classList.remove( stateBefore.pop() );
 		}
@@ -912,6 +955,7 @@ var Reveal = (function(){
 
 		updateControls();
 		updateProgress();
+
 	}
 
 	/**
@@ -928,6 +972,7 @@ var Reveal = (function(){
 	 * bounds.
 	 */
 	function updateSlides( selector, index ) {
+
 		// Select all slides and convert the NodeList result to
 		// an array
 		var slides = toArray( document.querySelectorAll( selector ) ),
@@ -1014,6 +1059,7 @@ var Reveal = (function(){
 	 * Updates the progress bar to reflect the current slide.
 	 */
 	function updateProgress() {
+
 		// Update progress if enabled
 		if( config.progress && dom.progress ) {
 
@@ -1055,12 +1101,14 @@ var Reveal = (function(){
 			dom.progressbar.style.width = ( pastCount / ( totalCount - 1 ) ) * window.innerWidth + 'px';
 
 		}
+
 	}
 
 	/**
 	 * Updates the state of all control/navigation arrows.
 	 */
 	function updateControls() {
+
 		if ( config.controls && dom.controls ) {
 
 			var routes = availableRoutes();
@@ -1085,6 +1133,7 @@ var Reveal = (function(){
 			if( routes.right || routes.down ) dom.controlsNext.forEach( function( el ) { el.classList.add( 'enabled' ); } );
 
 		}
+
 	}
 
 	/**
@@ -1093,6 +1142,7 @@ var Reveal = (function(){
 	 * @return {Object} containing four booleans: left/right/up/down
 	 */
 	function availableRoutes() {
+
 		var horizontalSlides = document.querySelectorAll( HORIZONTAL_SLIDES_SELECTOR ),
 			verticalSlides = document.querySelectorAll( VERTICAL_SLIDES_SELECTOR );
 
@@ -1102,12 +1152,14 @@ var Reveal = (function(){
 			up: indexv > 0,
 			down: indexv < verticalSlides.length - 1
 		};
+
 	}
 
 	/**
 	 * Reads the current URL (hash) and navigates accordingly.
 	 */
 	function readURL() {
+
 		var hash = window.location.hash;
 
 		// Attempt to parse the hash as either an index or name
@@ -1137,6 +1189,7 @@ var Reveal = (function(){
 
 			slide( h, v );
 		}
+
 	}
 
 	/**
@@ -1147,6 +1200,7 @@ var Reveal = (function(){
 	 * writing the hash
 	 */
 	function writeURL( delay ) {
+
 		if( config.history ) {
 
 			// Make sure there's never more than one timeout running
@@ -1172,6 +1226,7 @@ var Reveal = (function(){
 				window.location.hash = url;
 			}
 		}
+
 	}
 
 	/**
@@ -1185,6 +1240,7 @@ var Reveal = (function(){
 	 * @return {Object} { h: <int>, v: <int> }
 	 */
 	function getIndices( slide ) {
+
 		// By default, return the current indices
 		var h = indexh,
 			v = indexv;
@@ -1207,6 +1263,7 @@ var Reveal = (function(){
 		}
 
 		return { h: h, v: v };
+
 	}
 
 	/**
@@ -1216,6 +1273,7 @@ var Reveal = (function(){
 	 * false otherwise
 	 */
 	function nextFragment() {
+
 		// Vertical slides:
 		if( document.querySelector( VERTICAL_SLIDES_SELECTOR + '.present' ) ) {
 			var verticalFragments = document.querySelectorAll( VERTICAL_SLIDES_SELECTOR + '.present .fragment:not(.visible)' );
@@ -1240,6 +1298,7 @@ var Reveal = (function(){
 		}
 
 		return false;
+
 	}
 
 	/**
@@ -1249,6 +1308,7 @@ var Reveal = (function(){
 	 * false otherwise
 	 */
 	function previousFragment() {
+
 		// Vertical slides:
 		if( document.querySelector( VERTICAL_SLIDES_SELECTOR + '.present' ) ) {
 			var verticalFragments = document.querySelectorAll( VERTICAL_SLIDES_SELECTOR + '.present .fragment.visible' );
@@ -1273,46 +1333,57 @@ var Reveal = (function(){
 		}
 
 		return false;
+
 	}
 
 	/**
 	 * Cues a new automated slide if enabled in the config.
 	 */
 	function cueAutoSlide() {
+
 		clearTimeout( autoSlideTimeout );
 
 		// Cue the next auto-slide if enabled
 		if( autoSlide ) {
 			autoSlideTimeout = setTimeout( navigateNext, autoSlide );
 		}
+
 	}
 
 	function navigateLeft() {
+
 		// Prioritize hiding fragments
 		if( availableRoutes().left && isOverviewActive() || previousFragment() === false ) {
 			slide( indexh - 1 );
 		}
+
 	}
 
 	function navigateRight() {
+
 		// Prioritize revealing fragments
 		if( availableRoutes().right && isOverviewActive() || nextFragment() === false ) {
 			slide( indexh + 1 );
 		}
+
 	}
 
 	function navigateUp() {
+
 		// Prioritize hiding fragments
 		if( availableRoutes().up && isOverviewActive() || previousFragment() === false ) {
 			slide( indexh, indexv - 1 );
 		}
+
 	}
 
 	function navigateDown() {
+
 		// Prioritize revealing fragments
 		if( availableRoutes().down && isOverviewActive() || nextFragment() === false ) {
 			slide( indexh, indexv + 1 );
 		}
+
 	}
 
 	/**
@@ -1322,6 +1393,7 @@ var Reveal = (function(){
 	 * 3) Previous horizontal slide
 	 */
 	function navigatePrev() {
+
 		// Prioritize revealing fragments
 		if( previousFragment() === false ) {
 			if( availableRoutes().up ) {
@@ -1338,12 +1410,14 @@ var Reveal = (function(){
 				}
 			}
 		}
+
 	}
 
 	/**
 	 * Same as #navigatePrev() but navigates forwards.
 	 */
 	function navigateNext() {
+
 		// Prioritize revealing fragments
 		if( nextFragment() === false ) {
 			availableRoutes().down ? navigateDown() : navigateRight();
@@ -1352,6 +1426,7 @@ var Reveal = (function(){
 		// If auto-sliding is enabled we need to cue up
 		// another timeout
 		cueAutoSlide();
+
 	}
 
 
@@ -1366,6 +1441,7 @@ var Reveal = (function(){
 	 * @param {Object} event
 	 */
 	function onDocumentKeyDown( event ) {
+
 		// Check if there's a focused element that could be using
 		// the keyboard
 		var activeElement = document.activeElement;
@@ -1428,6 +1504,7 @@ var Reveal = (function(){
 	 * enables support for swipe and pinch gestures.
 	 */
 	function onDocumentTouchStart( event ) {
+
 		touch.startX = event.touches[0].clientX;
 		touch.startY = event.touches[0].clientY;
 		touch.startCount = event.touches.length;
@@ -1443,12 +1520,14 @@ var Reveal = (function(){
 				y: touch.startY
 			} );
 		}
+
 	}
 
 	/**
 	 * Handler for the document level 'touchmove' event.
 	 */
 	function onDocumentTouchMove( event ) {
+
 		// Each touch should only trigger one action
 		if( !touch.handled ) {
 			var currentX = event.touches[0].clientX;
@@ -1515,13 +1594,16 @@ var Reveal = (function(){
 		else if( navigator.userAgent.match( /android/gi ) ) {
 			event.preventDefault();
 		}
+
 	}
 
 	/**
 	 * Handler for the document level 'touchend' event.
 	 */
 	function onDocumentTouchEnd( event ) {
+
 		touch.handled = false;
+
 	}
 
 	/**
@@ -1529,6 +1611,7 @@ var Reveal = (function(){
 	 * multiple slides.
 	 */
 	function onDocumentMouseScroll( event ) {
+
 		clearTimeout( mouseWheelTimeout );
 
 		mouseWheelTimeout = setTimeout( function() {
@@ -1540,6 +1623,7 @@ var Reveal = (function(){
 				navigatePrev();
 			}
 		}, 100 );
+
 	}
 
 	/**
@@ -1549,30 +1633,37 @@ var Reveal = (function(){
 	 * ( clickX / presentationWidth ) * numberOfSlides
 	 */
 	function onProgressClick( event ) {
+
 		var slidesTotal = toArray( document.querySelectorAll( HORIZONTAL_SLIDES_SELECTOR ) ).length;
 		var slideIndex = Math.floor( ( event.clientX / dom.wrapper.offsetWidth ) * slidesTotal );
 
 		slide( slideIndex );
+
 	}
 
 	/**
 	 * Handler for the window level 'hashchange' event.
 	 */
 	function onWindowHashChange( event ) {
+
 		readURL();
+
 	}
 
 	/**
 	 * Handler for the window level 'resize' event.
 	 */
 	function onWindowResize( event ) {
+
 		layout();
+
 	}
 
 	/**
 	 * Invoked when a slide is and we're in the overview.
 	 */
 	function onOverviewSlideClicked( event ) {
+
 		// TODO There's a bug here where the event listeners are not
 		// removed after deactivating the overview.
 		if( isOverviewActive() ) {
@@ -1593,6 +1684,7 @@ var Reveal = (function(){
 				slide( h, v );
 			}
 		}
+
 	}
 
 
@@ -1624,6 +1716,9 @@ var Reveal = (function(){
 		navigatePrev: navigatePrev,
 		navigateNext: navigateNext,
 
+		// Forces an update in slide layout
+		layout: layout,
+
 		// Toggles the overview mode on/off
 		toggleOverview: toggleOverview,