fix read issue that caused double navigations fixes hakimel#3079

stefanocampanella · Mar 1, 2022 · 463f798 · 463f798
1 parent eea9443
commit 463f798
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 4 deletions.
diff --git a/dist/reveal.esm.js b/dist/reveal.esm.js
diff --git a/dist/reveal.js b/dist/reveal.js
diff --git a/js/controllers/location.js b/js/controllers/location.js
@@ -96,9 +96,13 @@ export default class Location {
 		const currentIndices = this.Reveal.getIndices();
 		const newIndices = this.getIndicesFromHash();
 
-		if( newIndices && ( newIndices.h !== currentIndices.h || newIndices.v !== currentIndices.v || newIndices.f !== undefined ) ) {
-			this.Reveal.slide( newIndices.h, newIndices.v, newIndices.f );
+		if( newIndices ) {
+			if( ( newIndices.h !== currentIndices.h || newIndices.v !== currentIndices.v || newIndices.f !== undefined ) ) {
+					this.Reveal.slide( newIndices.h, newIndices.v, newIndices.f );
+			}
 		}
+		// If no new indices are available, we're trying to navigate to
+		// a slide hash that does not exist
 		else {
 			this.Reveal.slide( currentIndices.h || 0, currentIndices.v || 0 );
 		}