add leftRightSkipFragments config

Author: dvirtz

.gitignore

@@ -9,4 +9,5 @@ log/*.log
 tmp/**
 node_modules/
 .sass-cache
-dist/*.map
+dist/*.map
+.vscode/

dist/reveal.esm.js

@@ -122,6 +122,10 @@ export default {
 	// from 1.3 -> 2.3.
 	navigationMode: 'default',
 
+	// Make left/right arrow keys skip current slide's fragment
+	// and move directly to previous/next slide
+	leftRightSkipFragments: false,
+
 	// Randomizes the order of slides each time the presentation loads
 	shuffle: false,
 

dist/reveal.js

@@ -2157,12 +2157,12 @@ export default function( revealElement, options ) {
 
 		// Reverse for RTL
 		if( config.rtl ) {
-			if( ( overview.isActive() || fragments.next() === false ) && availableRoutes().left ) {
+			if( ( overview.isActive() || config.leftRightSkipFragments || fragments.next() === false ) && availableRoutes().left ) {
 				slide( indexh + 1, config.navigationMode === 'grid' ? indexv : undefined );
 			}
 		}
 		// Normal navigation
-		else if( ( overview.isActive() || fragments.prev() === false ) && availableRoutes().left ) {
+		else if( ( overview.isActive() || config.leftRightSkipFragments || fragments.prev() === false ) && availableRoutes().left ) {
 			slide( indexh - 1, config.navigationMode === 'grid' ? indexv : undefined );
 		}
 
@@ -2174,12 +2174,12 @@ export default function( revealElement, options ) {
 
 		// Reverse for RTL
 		if( config.rtl ) {
-			if( ( overview.isActive() || fragments.prev() === false ) && availableRoutes().right ) {
+			if( ( overview.isActive() || config.leftRightSkipFragments || fragments.prev() === false ) && availableRoutes().right ) {
 				slide( indexh - 1, config.navigationMode === 'grid' ? indexv : undefined );
 			}
 		}
 		// Normal navigation
-		else if( ( overview.isActive() || fragments.next() === false ) && availableRoutes().right ) {
+		else if( ( overview.isActive() || config.leftRightSkipFragments || fragments.next() === false ) && availableRoutes().right ) {
 			slide( indexh + 1, config.navigationMode === 'grid' ? indexv : undefined );
 		}
 

js/config.js

@@ -505,6 +505,28 @@ <h1>4</h1>
 					assert.deepEqual( Reveal.getIndices(), { h: 2, v: 0, f: 0 }, 'up() goes to prev fragment' );
 				});
 
+				QUnit.test( 'Skipping fragments', function( assert ) {
+					Reveal.configure({ leftRightSkipFragments: true });
+
+					// forwards:
+
+					Reveal.slide( 2, 0, -1 );
+
+					Reveal.right();
+					assert.deepEqual( Reveal.getIndices(), { h: 3, v: 0, f: undefined }, 'when leftRightSkipFragments is set, right() goes to next slide' );
+
+					// backwards:
+
+					Reveal.left(); // return to previous slide
+					assert.deepEqual( Reveal.getIndices(), { h: 2, v: 0, f: 2 }, 'left() goes to prev fragment' );
+
+					Reveal.left();
+					assert.deepEqual( Reveal.getIndices(), { h: 1, v: 0, f: undefined }, 'when leftRightSkipFragments is set, left() goes to prev slide' );
+
+					Reveal.configure({ leftRightSkipFragments: false });
+				});
+
+
 				QUnit.test( 'Stepping past fragments', function( assert ) {
 					var fragmentSlide = document.querySelector( '#fragment-slides>section:nth-child(1)' );