don't start video bgs if autoPlayMedia config is set to false

hakimel · Mar 11, 2024 · 62b1ea3 · 62b1ea3
1 parent 76ec60a
commit 62b1ea3
Show file tree

Hide file tree

Showing 5 changed files with 10 additions and 8 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/backgrounds.js b/js/controllers/backgrounds.js
@@ -268,14 +268,15 @@ export default class Backgrounds {
 	 */
 	update( includeAll = false ) {
 
+		let config = this.Reveal.getConfig();
 		let currentSlide = this.Reveal.getCurrentSlide();
 		let indices = this.Reveal.getIndices();
 
 		let currentBackground = null;
 
 		// Reverse past/future classes when in RTL mode
-		let horizontalPast = this.Reveal.getConfig().rtl ? 'future' : 'past',
-			horizontalFuture = this.Reveal.getConfig().rtl ? 'past' : 'future';
+		let horizontalPast = config.rtl ? 'future' : 'past',
+			horizontalFuture = config.rtl ? 'past' : 'future';
 
 		// Update the classes of all backgrounds to match the
 		// states of their slides (past/present/future)
@@ -373,8 +374,9 @@ export default class Backgrounds {
 						previousVideoParent.appendChild( currentVideo );
 						currentVideoParent.appendChild( previousVideo );
 
-						// Resume playing if the previous video was playing
-						previousVideo.play();
+						if( config.autoPlayMedia !== false ) {
+							previousVideo.play();
+						}
 					}
 				}
 			}