Merge pull request #2251 from oyron/2229-speaker-notes-bug
Give focus to previously opened speaker notes (fixes #2229)
This commit is contained in:
		| @@ -11,15 +11,22 @@ | |||||||
|  */ |  */ | ||||||
| var RevealNotes = (function() { | var RevealNotes = (function() { | ||||||
|  |  | ||||||
|  |     var notesPopup = null; | ||||||
|  |  | ||||||
| 	function openNotes( notesFilePath ) { | 	function openNotes( notesFilePath ) { | ||||||
|  |  | ||||||
|  |         if (notesPopup && !notesPopup.closed) { | ||||||
|  |             notesPopup.focus(); | ||||||
|  |             return; | ||||||
|  |         } | ||||||
|  |  | ||||||
| 		if( !notesFilePath ) { | 		if( !notesFilePath ) { | ||||||
| 			var jsFileLocation = document.querySelector('script[src$="notes.js"]').src;  // this js file path | 			var jsFileLocation = document.querySelector('script[src$="notes.js"]').src;  // this js file path | ||||||
| 			jsFileLocation = jsFileLocation.replace(/notes\.js(\?.*)?$/, '');   // the js folder path | 			jsFileLocation = jsFileLocation.replace(/notes\.js(\?.*)?$/, '');   // the js folder path | ||||||
| 			notesFilePath = jsFileLocation + 'notes.html'; | 			notesFilePath = jsFileLocation + 'notes.html'; | ||||||
| 		} | 		} | ||||||
|  |  | ||||||
| 		var notesPopup = window.open( notesFilePath, 'reveal.js - Notes', 'width=1100,height=700' ); | 		notesPopup = window.open( notesFilePath, 'reveal.js - Notes', 'width=1100,height=700' ); | ||||||
|  |  | ||||||
| 		if( !notesPopup ) { | 		if( !notesPopup ) { | ||||||
| 			alert( 'Speaker view popup failed to open. Please make sure popups are allowed and reopen the speaker view.' ); | 			alert( 'Speaker view popup failed to open. Please make sure popups are allowed and reopen the speaker view.' ); | ||||||
|   | |||||||
		Reference in New Issue
	
	Block a user