From 8e66876c4e5881d3a85516e0070094bc8b0d8b9f Mon Sep 17 00:00:00 2001 From: Hakim El Hattab Date: Thu, 29 Jan 2015 12:21:05 +0100 Subject: [PATCH] fix error when exiting overview --- js/reveal.js | 11 +++++------ 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/js/reveal.js b/js/reveal.js index ac21f33..ad2d5a3 100644 --- a/js/reveal.js +++ b/js/reveal.js @@ -1074,8 +1074,8 @@ function transformSlides( transforms ) { // Pick up new transforms from arguments - if( transforms.layout ) slidesTransform.layout = transforms.layout; - if( transforms.overview ) slidesTransform.overview = transforms.overview; + if( typeof transforms.layout === 'string' ) slidesTransform.layout = transforms.layout; + if( typeof transforms.overview === 'string' ) slidesTransform.overview = transforms.overview; // Apply the transforms to the slides container if( slidesTransform.layout ) { @@ -1085,7 +1085,6 @@ transformElement( dom.slides, slidesTransform.overview ); } - } /** @@ -1780,8 +1779,6 @@ // Only proceed if enabled in config if( config.overview ) { - transformSlides( { overview: '' } ); - overview = false; dom.wrapper.classList.remove( 'overview' ); @@ -1811,6 +1808,8 @@ transformElement( background, '' ); } ); + transformSlides( { overview: '' } ); + slide( indexh, indexv ); layout(); @@ -2053,7 +2052,7 @@ document.documentElement.classList.remove( stateBefore.pop() ); } - // If the overview is active, re-activate it to update positions + // Update the overview if it's currently active if( isOverview() ) { updateOverview(); }