notes plugin no longer syncs overview mode hakimel#1446

SteveDesmond-ca · Mar 9, 2016 · 8b21e87 · 8b21e87
1 parent 839398c
commit 8b21e87
Showing 1 changed file with 7 additions and 1 deletion.
diff --git a/plugin/notes/notes.html b/plugin/notes/notes.html
@@ -188,6 +188,10 @@ <h4 class="label">Notes</h4>
 
 					var data = JSON.parse( event.data );
 
+					// The overview mode is only useful to the reveal.js instance
+					// where navigation occurs so we don't sync it
+					if( data.state ) delete data.state.overview;
+
 					// Messages sent by the notes plugin inside of the main window
 					if( data && data.namespace === 'reveal-notes' ) {
 						if( data.type === 'connect' ) {
@@ -203,8 +207,10 @@ <h4 class="label">Notes</h4>
 							// Send a message back to notify that the handshake is complete
 							window.opener.postMessage( JSON.stringify({ namespace: 'reveal-notes', type: 'connected'} ), '*' );
 						}
-						else if( /slidechanged|fragmentshown|fragmenthidden|overviewshown|overviewhidden|paused|resumed/.test( data.eventName ) && currentState !== JSON.stringify( data.state ) ) {
+						else if( /slidechanged|fragmentshown|fragmenthidden|paused|resumed/.test( data.eventName ) && currentState !== JSON.stringify( data.state ) ) {
+
 							window.opener.postMessage( JSON.stringify({ method: 'setState', args: [ data.state ]} ), '*' );
+
 						}
 					}