Merge pull request #2263 from mbotsch/fix-readURL

Fix if-statement in readURL
hakimel · Oct 25, 2018 · 4ac153c · 4ac153c
2 parents 7d66999 + 8a35a1e
commit 4ac153c
Showing 1 changed file with 7 additions and 4 deletions.
diff --git a/js/reveal.js b/js/reveal.js
@@ -4070,10 +4070,13 @@
 			// Ensure that we're not already on a slide with the same name
 			var isSameNameAsCurrentSlide = currentSlide ? currentSlide.getAttribute( 'id' ) === name : false;
 
-			if( element && !isSameNameAsCurrentSlide ) {
-				// Find the position of the named slide and navigate to it
-				var indices = Reveal.getIndices( element );
-				slide( indices.h, indices.v );
+			if( element ) {
+				// If the slide exists and is not the current slide...
+				if ( !isSameNameAsCurrentSlide ) {
+					// ...find the position of the named slide and navigate to it
+					var indices = Reveal.getIndices(element);
+					slide(indices.h, indices.v);
+				}
 			}
 			// If the slide doesn't exist, navigate to the current slide
 			else {