Bladeren bron

formatting and tweaks for #2104

Hakim El Hattab 5 jaren geleden
bovenliggende
commit
d5cf3fa13c
4 gewijzigde bestanden met toevoegingen van 23 en 34 verwijderingen
  1. 3 20
      index.html
  2. 10 8
      js/reveal.js
  3. 5 3
      plugin/notes/notes.html
  4. 5 3
      plugin/notes/notes.js

+ 3 - 20
index.html

@@ -24,23 +24,8 @@
 	<body>
 		<div class="reveal">
 			<div class="slides">
-				<section data-timing="6">
-					Slide 1
-
-					<aside>
-						these are not notes
-					</aside>
-					<aside class="notes">
-						these are notes for the first slide
-					</aside>
-				</section>
-				<section>
-					Slide 2
-
-					<aside class="notes">
-						these are the nodes for the second slide
-					</aside>
-				</section>
+				<section>Slide 1</section>
+				<section>Slide 2</section>
 			</div>
 		</div>
 
@@ -57,9 +42,7 @@
 					{ src: 'plugin/markdown/markdown.js' },
 					{ src: 'plugin/notes/notes.js', async: true },
 					{ src: 'plugin/highlight/highlight.js', async: true, callback: function() { hljs.initHighlightingOnLoad(); } }
-				],
-				controlsTutorial: false,
-				defaultTiming: 3
+				]
 			});
 		</script>
 	</body>

+ 10 - 8
js/reveal.js

@@ -4099,18 +4099,20 @@
 	}
 
 	/**
-	 * Returns an array of objects where each object represents the attributes on its respective slide.
+	 * Returns an array of objects where each object represents the
+	 * attributes on its respective slide.
 	 */
-	function getSlidesMetaInfo() {
+	function getSlidesAttributes() {
 
-		var slides = getSlides();
-		return slides.map( function (slide) {
-			var meta = {};
+		return getSlides().map( function( slide ) {
+
+			var attributes = {};
 			for( var i = 0; i < slide.attributes.length; i++ ) {
 				var attribute = slide.attributes[ i ];
-				meta[ attribute.name ] = attribute.value;
+				attributes[ attribute.name ] = attribute.value;
 			}
-			return meta;
+			return attributes;
+
 		} );
 
 	}
@@ -5473,7 +5475,7 @@
 
 		// Returns an Array of objects representing the attributes on
 		// the slides
-		getSlidesMetaInfo: getSlidesMetaInfo,
+		getSlidesAttributes: getSlidesAttributes,
 
 		// Returns the total number of slides
 		getTotalSlides: getTotalSlides,

+ 5 - 3
plugin/notes/notes.html

@@ -408,6 +408,7 @@
 				 * Asynchronously calls the Reveal.js API of the main frame.
 				 */
 				function callRevealApi( methodName, methodArguments, callback ) {
+
 					var callId = ++lastRevealApiCallId;
 					pendingCalls[callId] = callback;
 					window.opener.postMessage( JSON.stringify( {
@@ -417,6 +418,7 @@
 						methodName: methodName,
 						arguments: methodArguments
 					} ), '*' );
+
 				}
 
 				/**
@@ -535,7 +537,7 @@
 
 				function getTimings( callback ) {
 
-					callRevealApi( 'getSlidesMetaInfo', [], function ( slides ) {
+					callRevealApi( 'getSlidesAttributes', [], function ( slideAttributes ) {
 						callRevealApi( 'getConfig', [], function ( config ) {
 							var defaultTiming = config.defaultTiming;
 							if (defaultTiming == null) {
@@ -544,8 +546,8 @@
 							}
 
 							var timings = [];
-							for ( var i in slides ) {
-								var slide = slides[ i ];
+							for ( var i in slideAttributes ) {
+								var slide = slideAttributes[ i ];
 								var timing = defaultTiming;
 								if( slide.hasOwnProperty( 'data-timing' )) {
 									var t = slide[ 'data-timing' ];

+ 5 - 3
plugin/notes/notes.js

@@ -56,17 +56,19 @@ var RevealNotes = (function() {
 		}
 
 		/**
-		 * Calls the specified Reveal.js method with the provided argument and then pushes the result to the notes
-		 * frame.
+		 * Calls the specified Reveal.js method with the provided argument
+		 * and then pushes the result to the notes frame.
 		 */
 		function callRevealApi( methodName, methodArguments, callId ) {
-			var result = Reveal[methodName].call(Reveal, methodArguments);
+
+			var result = Reveal[methodName].call( Reveal, methodArguments );
 			notesPopup.postMessage( JSON.stringify( {
 				namespace: 'reveal-notes',
 				type: 'return',
 				result: result,
 				callId: callId
 			} ), '*' );
+
 		}
 
 		/**