fix broken mobile scroll view navigation where there were fragments s…

…tarting at an index above 1 hakimel#3540
boycaught · Nov 30, 2023 · d5896c9 · d5896c9
1 parent bf285af
commit d5896c9
Show file tree

Hide file tree

Showing 5 changed files with 14 additions and 14 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
@@ -425,7 +425,6 @@ export default class ScrollView {
 			];
 
 			const scrollTriggerSegmentSize = ( trigger.range[1] - trigger.range[0] ) / trigger.page.scrollTriggers.length;
-
 			// Set the range for each inner scroll trigger
 			trigger.page.scrollTriggers.forEach( ( scrollTrigger, i ) => {
 				scrollTrigger.range = [
@@ -462,16 +461,17 @@ export default class ScrollView {
 					activate: () => {
 						this.Reveal.fragments.update( -1, page.fragments, slideElement );
 					}
-				},
-
-				// Triggers for each fragment group
-				...fragmentGroups.map( ( fragments, i ) => ({
-						activate: () => {
-							this.Reveal.fragments.update( i, page.fragments, slideElement );
-						}
-					})
-				)
+				}
 			);
+
+			// Triggers for each fragment group
+			fragmentGroups.forEach( ( fragments, i ) => {
+				page.scrollTriggers.push({
+					activate: () => {
+						this.Reveal.fragments.update( i, page.fragments, slideElement );
+					}
+				});
+			} );
 		}