Browse Source

prevent default keyboard shortcuts while overlay is open #3766 #3767

Hakim El Hattab 1 month ago
parent
commit
b3fd27d071
5 changed files with 8 additions and 0 deletions
  1. 0 0
      dist/reveal.esm.js
  2. 0 0
      dist/reveal.esm.js.map
  3. 0 0
      dist/reveal.js
  4. 0 0
      dist/reveal.js.map
  5. 8 0
      js/controllers/keyboard.js

File diff suppressed because it is too large
+ 0 - 0
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
+ 0 - 0
dist/reveal.js


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


+ 8 - 0
js/controllers/keyboard.js

@@ -190,6 +190,10 @@ export default class Keyboard {
 			}
 			}
 		}
 		}
 
 
+		if( this.Reveal.isOverlayOpen() && !['Escape', 'f', 'c'].includes(event.key) ) {
+			return false;
+		}
+
 		if( this.Reveal.isPaused() && resumeKeyCodes.indexOf( keyCode ) === -1 ) {
 		if( this.Reveal.isPaused() && resumeKeyCodes.indexOf( keyCode ) === -1 ) {
 			return false;
 			return false;
 		}
 		}
@@ -363,6 +367,10 @@ export default class Keyboard {
 					this.Reveal.toggleJumpToSlide();
 					this.Reveal.toggleJumpToSlide();
 				}
 				}
 			}
 			}
+			// C
+			else if( keyCode === 67 && this.Reveal.isOverlayOpen() ) {
+				this.Reveal.closeOverlay();
+			}
 			// ?
 			// ?
 			else if( ( keyCode === 63 || keyCode === 191 ) && event.shiftKey ) {
 			else if( ( keyCode === 63 || keyCode === 191 ) && event.shiftKey ) {
 				this.Reveal.toggleHelp();
 				this.Reveal.toggleHelp();

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