From 233160ff2325037a750cfc8153f5c82ac0811219 Mon Sep 17 00:00:00 2001 From: Hakim El Hattab Date: Sun, 4 Dec 2011 18:55:48 -0800 Subject: [PATCH] cleanup & comments --- index.html | 26 ++++---------------------- js/reveal.js | 19 +++++++++---------- 2 files changed, 13 insertions(+), 32 deletions(-) diff --git a/index.html b/index.html index c9b70bd..a1f7e6e 100644 --- a/index.html +++ b/index.html @@ -100,7 +100,8 @@

Pretty Code

-
var supports3DTransforms =  document.body.style['webkitPerspective'] !== undefined || 
+				

+var supports3DTransforms =  document.body.style['webkitPerspective'] !== undefined || 
 				document.body.style['MozPerspective'] !== undefined ||
 				document.body.style['perspective'] !== undefined;
 
@@ -120,7 +121,8 @@ function linkify( selector ) {
     }
 }
 
-linkify( 'a' );
+linkify( 'a' ); +

Courtesy of highlight.js.

@@ -191,25 +193,5 @@ linkify( 'a' ); - - - -
- - - -
- - - \ No newline at end of file diff --git a/js/reveal.js b/js/reveal.js index 6614493..6ce7a89 100644 --- a/js/reveal.js +++ b/js/reveal.js @@ -25,9 +25,7 @@ * sexy CSS 3D transforms. * * Slides are given unique hash based URL's so that they can be - * opened directly. I didn't use the HTML5 History API for this - * as it would have required the addition of server side rewrite - * rules and hence require more effort for anyone to set up. + * opened directly. * * Public facing methods: * - Reveal.initialize( { ... options ... } ); @@ -48,9 +46,7 @@ * * version 0.3: * - Added licensing terms - * - * version 0.4: - * - Fixed broken links on touch devices. + * - Fixed broken links on touch devices * * version 1.0: * - Added controls @@ -63,15 +59,16 @@ * * TODO: * - Touch/swipe interactions + * - Presentation overview via keyboard shortcut * - * @author Hakim El Hattab + * @author Hakim El Hattab | http://hakim.se * @version 1.0 */ var Reveal = (function(){ var HORIZONTAL_SLIDES_SELECTOR = '#main>section', VERTICAL_SLIDES_SELECTOR = 'section.present>section', - + indexh = 0, indexv = 0, @@ -98,7 +95,7 @@ var Reveal = (function(){ dom.controlsUp.addEventListener('click', preventAndForward( navigateUp ), false); dom.controlsDown.addEventListener('click', preventAndForward( navigateDown ), false); - // Set default configuration + // Default options config.rollingLinks = options.rollingLinks === undefined ? true : options.rollingLinks; config.controls = options.controls === undefined ? false : options.controls; config.theme = options.theme === undefined ? 'default' : options.theme; @@ -116,7 +113,7 @@ var Reveal = (function(){ linkify(); } - // Read the initial state of the URL (hash) + // Read the initial hash readURL(); } @@ -310,7 +307,9 @@ var Reveal = (function(){ } /** + * Determine what available routes there are for navigation. * + * @return {Object} containing four booleans: left/right/up/down */ function availableRoutes() { var horizontalSlides = document.querySelectorAll( HORIZONTAL_SLIDES_SELECTOR ); -- 2.26.2