From 902e36c0228478083d9f2ef487c12f9de401d404 Mon Sep 17 00:00:00 2001
From: Hakim El Hattab <hakim.elhattab@gmail.com>
Date: Sun, 27 Apr 2014 14:54:23 +0200
Subject: [PATCH] break showing/hiding of slides into separate methods
---
js/reveal.js | 29 ++++++++++++++++++++---------
1 file changed, 20 insertions(+), 9 deletions(-)
diff --git a/js/reveal.js b/js/reveal.js
index 55642e2..2c87c87 100644
--- a/js/reveal.js
+++ b/js/reveal.js
@@ -2029,11 +2029,10 @@ var Reveal = (function(){
// Show the horizontal slide if it's within the view distance
if( distanceX < viewDistance ) {
- horizontalSlide.style.display = 'block';
- loadSlide( horizontalSlide );
+ showSlide( horizontalSlide );
}
else {
- horizontalSlide.style.display = 'none';
+ hideSlide( horizontalSlide );
}
if( verticalSlidesLength ) {
@@ -2046,11 +2045,10 @@ var Reveal = (function(){
distanceY = x === indexh ? Math.abs( indexv - y ) : Math.abs( y - oy );
if( distanceX + distanceY < viewDistance ) {
- verticalSlide.style.display = 'block';
- loadSlide( verticalSlide );
+ showSlide( verticalSlide );
}
else {
- verticalSlide.style.display = 'none';
+ hideSlide( verticalSlide );
}
}
@@ -2275,10 +2273,13 @@ var Reveal = (function(){
}
/**
- * Loads any content that is set to load lazily (data-src)
- * inside of the given slide.
+ * Called when the given slide is within the configured view
+ * distance. Shows the slide element and loads any content
+ * that is set to load lazily (data-src).
*/
- function loadSlide( slide ) {
+ function showSlide( slide ) {
+
+ slide.style.display = 'block';
// Media elements with data-src attributes
toArray( slide.querySelectorAll( 'img[data-src], video[data-src], audio[data-src], iframe[data-src]' ) ).forEach( function( element ) {
@@ -2305,6 +2306,16 @@ var Reveal = (function(){
}
+ /**
+ * Called when the given slide is moved outside of the
+ * configured view distance.
+ */
+ function hideSlide( slide ) {
+
+ slide.style.display = 'none';
+
+ }
+
/**
* Determine what available routes there are for navigation.
*
--
GitLab