null check progress DOM element even if it's configured to be on

diff --git a/README.md b/README.md
index acc41477915eeb9cc9358eda8f5c3464be4bb02f..298c1cfd78d484a5fdaa8aae4d01fc816c3fd8cc 100644 (file)
--- a/README.md
+++ b/README.md
@@ -176,6 +176,7 @@ You can change the appearance of the speaker notes by editing the file at `plugi
 - Added beige theme
 - Added 'autoSlide' config
 - Bug fix: The 'slidechanged' event is now firing upon 'hashchange'. Thanks [basecode](https://github.com/basecode)
+- Bug fix: JS error when the 'progress' option was true but there was no progress DOM element
 
 #### 1.3
 - Revised keyboard shortcuts, including ESC for overview, N for next, P for previous. Thanks [mahemoff](https://github.com/mahemoff)

diff --git a/js/reveal.js b/js/reveal.js
index c9e69a8f8b372e5688a18bcc7a9b6c96ea69ff13..e0be83907017a34107c6d9e93456f0c9c33a74f9 100644 (file)
--- a/js/reveal.js
+++ b/js/reveal.js
@@ -155,7 +155,7 @@ var Reveal = (function(){
                        dom.controls.style.display = 'block';
                }
 
-               if( config.progress ) {
+               if( config.progress && dom.progress ) {
                        dom.progress.style.display = 'block';
                }
 
@@ -680,7 +680,7 @@ var Reveal = (function(){
                }
 
                // Update progress if enabled
-               if( config.progress ) {
+               if( config.progress && dom.progress ) {
                        dom.progressbar.style.width = ( indexh / ( document.querySelectorAll( HORIZONTAL_SLIDES_SELECTOR ).length - 1 ) ) * window.innerWidth + 'px';
                }