From 8a35a1e1ed9b0579f1e549273348a0477167c207 Mon Sep 17 00:00:00 2001 From: Mario Botsch Date: Wed, 24 Oct 2018 23:22:11 +0200 Subject: [PATCH] Fix nested if-statement in readURL that caused slide(h,v) to be called even when the current slide is the target slide. --- js/reveal.js | 11 +++++++---- 1 file changed, 7 insertions(+), 4 deletions(-) diff --git a/js/reveal.js b/js/reveal.js index d78d3a92..f7970844 100644 --- 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 {