fix last slide not triggering slidechange in scroll view (closes #3715)

hakimel · Nov 11, 2024 · fe4a6e8 · fe4a6e8
1 parent 5a275f2
commit fe4a6e8
Show file tree

Hide file tree

Showing 5 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.esm.js.map b/dist/reveal.esm.js.map
diff --git a/dist/reveal.js b/dist/reveal.js
diff --git a/dist/reveal.js.map b/dist/reveal.js.map
diff --git a/js/controllers/scrollview.js b/js/controllers/scrollview.js
@@ -449,6 +449,10 @@ export default class ScrollView {
 			rangeStart = trigger.range[1];
 		} );
 
+		// Ensure the last trigger extends to the end of the page, otherwise
+		// rounding errors can cause the last trigger to end at 0.999999...
+		this.slideTriggers[this.slideTriggers.length - 1].range[1] = 1;
+
 	}
 
 	/**