Browse Source

fix issue where background of a future vertical slide is briefly visible ahead of time #3520

Hakim El Hattab 1 year ago
parent
commit
1e1e228680
5 changed files with 6 additions and 4 deletions
  1. 1 1
      dist/reveal.esm.js
  2. 0 0
      dist/reveal.esm.js.map
  3. 1 1
      dist/reveal.js
  4. 0 0
      dist/reveal.js.map
  5. 4 2
      js/controllers/backgrounds.js

File diff suppressed because it is too large
+ 1 - 1
dist/reveal.esm.js


File diff suppressed because it is too large
+ 0 - 0
dist/reveal.esm.js.map


File diff suppressed because it is too large
+ 1 - 1
dist/reveal.js


File diff suppressed because it is too large
+ 0 - 0
dist/reveal.js.map


+ 4 - 2
js/controllers/backgrounds.js

@@ -301,10 +301,12 @@ export default class Backgrounds {
 
 					backgroundv.classList.remove( 'past', 'present', 'future' );
 
-					if( v < indices.v ) {
+					const indexv = typeof indices.v === 'number' ? indices.v : 0;
+
+					if( v < indexv ) {
 						backgroundv.classList.add( 'past' );
 					}
-					else if ( v > indices.v ) {
+					else if ( v > indexv ) {
 						backgroundv.classList.add( 'future' );
 					}
 					else {

Some files were not shown because too many files changed in this diff