From: Michael Kühnel <mail@michael-kuehnel.de>
Date: Wed, 24 Oct 2012 13:06:32 +0000 (+0200)
Subject: Delete functionality to control presentation from notes window
X-Git-Tag: 2.1.0~11^2
X-Git-Url: http://git.tremily.us/?a=commitdiff_plain;h=1801bf67eaf1fac8cb5776fe9ba83cc9e13272d6;p=reveal.js.git

Delete functionality to control presentation from notes window

Its was impossible (at least for me) to keep the windows in sync without bloating the code too much.
---

diff --git a/plugin/notes/notes.html b/plugin/notes/notes.html
index 775ccb4..90083cf 100644
--- a/plugin/notes/notes.html
+++ b/plugin/notes/notes.html
@@ -119,9 +119,6 @@
 							}
 						}
 
-						// Kill the slide listeners while responding to the event
-						removeSlideListeners();
-
 						// Update the note slides
 						currentSlide.contentWindow.Reveal.slide( data.indexh, data.indexv );
 						nextSlide.contentWindow.Reveal.slide( data.nextindexh, data.nextindexv );
@@ -134,30 +131,8 @@
 							currentSlide.contentWindow.Reveal.prevFragment();
 						}
 
-						// Resume listening on the next cycle
-						setTimeout( addSlideListeners, 1 );
-
 					}, false );
 
-					function addSlideListeners() {
-						currentSlide.contentWindow.Reveal.addEventListener( 'slidechanged', onNotesSlideChange, false );
-						nextSlide.contentWindow.Reveal.addEventListener( 'slidechanged', onNotesSlideChange, false );
-					}
-
-					function removeSlideListeners() {
-						currentSlide.contentWindow.Reveal.removeEventListener( 'slidechanged', onNotesSlideChange, false );
-						nextSlide.contentWindow.Reveal.removeEventListener( 'slidechanged', onNotesSlideChange, false );
-					}
-
-					function onNotesSlideChange( event ) {
-						window.opener.postMessage( JSON.stringify({
-							indexh : event.indexh,
-							indexv : event.indexv
-						}), '*' );
-					}
-
-					addSlideListeners();
-
 				})( window );
 
 			}, false );
diff --git a/plugin/notes/notes.js b/plugin/notes/notes.js
index 0573440..00d386d 100644
--- a/plugin/notes/notes.js
+++ b/plugin/notes/notes.js
@@ -69,16 +69,6 @@ var RevealNotes = (function() {
 			notesPopup.postMessage( JSON.stringify( slideData ), '*' );
 		}
 
-		// The main presentation is kept in sync when navigating the
-		// note slides so that the popup may be used as a remote
-		window.addEventListener( 'message', function( event ) {
-			var data = JSON.parse( event.data );
-
-			if( data && typeof data.indexh === 'number' && typeof data.indexv === 'number' ) {
-				Reveal.slide( data.indexh, data.indexv );
-			}
-		} );
-
 		// Navigate to the current slide when the notes are loaded
 		notesPopup.addEventListener( 'load', function( event ) {
 			post('slidechanged');