Browse Source

fix padding and duplicate background on vertical slides in overview

Hakim El Hattab 12 years ago
parent
commit
1ac6386eef
2 changed files with 11 additions and 7 deletions
  1. 8 1
      css/main.css
  2. 3 6
      js/reveal.js

+ 8 - 1
css/main.css

@@ -261,12 +261,19 @@ h1 {
 }
 
 
+/*********************************************
+ * OVERVIEW
+ *********************************************/
+
 .overview #main section {
-	padding: 20px;
+	padding: 20px 0;
 	opacity: 1;
 	cursor: pointer;
 	background: rgba(0,0,0,0.1);
 }
+.overview #main section.present>section.present {
+	background: none;
+}
 .overview #main section>section {
 	opacity: 1;
 	cursor: pointer;

+ 3 - 6
js/reveal.js

@@ -118,8 +118,6 @@ var Reveal = (function(){
 		dom.controlsUp.addEventListener('click', preventAndForward( navigateUp ), false);
 		dom.controlsDown.addEventListener('click', preventAndForward( navigateDown ), false);
 
-
-
 		// Fall back on default options
 		config.rollingLinks = options.rollingLinks === undefined ? true : options.rollingLinks;
 		config.controls = options.controls === undefined ? false : options.controls;
@@ -172,7 +170,6 @@ var Reveal = (function(){
 	 * @param {Object} event
 	 */
 	function onDocumentKeyDown( event ) {
-		
 		// FFT: Use document.querySelector( ':focus' ) === null 
 		// instead of checking contentEditable?
 
@@ -247,11 +244,9 @@ var Reveal = (function(){
 			}
 			
 			slide();
-			
 		}
 	}
 	
-	
 	/**
 	 * Handler for the window level 'hashchange' event.
 	 * 
@@ -410,7 +405,9 @@ var Reveal = (function(){
 
 				// Optimization; hide all slides that are three or more steps 
 				// away from the present slide
-				// slide.style.display = Math.abs( index - i ) > 3 ? 'none' : 'block';
+				if( overviewIsActive() === false ) {
+					slide.style.display = Math.abs( index - i ) > 3 ? 'none' : 'block';
+				}
 
 				if( i < index ) {
 					// Any element previous to index is given the 'past' class