From: Hakim El Hattab <hakim.elhattab@gmail.com>
Date: Wed, 21 Aug 2013 02:32:19 +0000 (-0400)
Subject: merge support for notes when using external markdown
X-Git-Url: http://git.tremily.us/?a=commitdiff_plain;h=631a237b3645627407cb433514042611824d890f;p=reveal.js.git

merge support for notes when using external markdown
---

631a237b3645627407cb433514042611824d890f
diff --cc README.md
index a8fcaeb,e2e50c6..56df5a4
--- a/README.md
+++ b/README.md
@@@ -52,10 -52,10 +52,10 @@@ This is based on [data-markdown](https:
  
  #### External Markdown
  
 -You can write your content as a separate file and have reveal.js load it at runtime. Note the separator arguments which determine how slides are delimited in the external file.
 +You can write your content as a separate file and have reveal.js load it at runtime. Note the separator arguments which determine how slides are delimited in the external file. The ```data-charset``` attribute is optional and specifies which charset to use when loading the external file.
  
  ```html
- <section data-markdown="example.md" data-separator="^\n\n\n" data-vertical="^\n\n" data-charset="iso-8859-15"></section>
 -<section data-markdown="example.md" data-separator="^\n\n\n" data-vertical="^\n\n" data-notes="^[[["></section>
++<section data-markdown="example.md" data-separator="^\n\n\n" data-vertical="^\n\n" data-notes="^Note:" data-charset="iso-8859-15"></section>
  ```
  
  ### Configuration
diff --cc plugin/markdown/example.html
index ceed53f,1253cba..0b07aa5
--- a/plugin/markdown/example.html
+++ b/plugin/markdown/example.html
@@@ -87,7 -87,7 +87,8 @@@
  				dependencies: [
  					{ src: '../../lib/js/classList.js', condition: function() { return !document.body.classList; } },
  					{ src: 'marked.js', condition: function() { return !!document.querySelector( '[data-markdown]' ); } },
--					{ src: 'markdown.js', condition: function() { return !!document.querySelector( '[data-markdown]' ); } }
++                    { src: 'markdown.js', condition: function() { return !!document.querySelector( '[data-markdown]' ); } },
++					{ src: '../notes/notes.js' }
  				]
  			});
  
diff --cc plugin/markdown/example.md
index e988dd9,e988dd9..6f6f577
--- a/plugin/markdown/example.md
+++ b/plugin/markdown/example.md
@@@ -6,6 -6,6 +6,8 @@@
  
  Content 1.1
  
++Note: This will only appear in the speaker notes window.
++
  
  ## External 1.2
  
diff --cc plugin/markdown/markdown.js
index a5abc30,8366c50..bc771e7
--- a/plugin/markdown/markdown.js
+++ b/plugin/markdown/markdown.js
@@@ -36,7 -28,9 +36,9 @@@
      };
  
      var twrap = function(el) {
-       return marked(el);
 -      var cnt = el.content || el;
 -      cnt += el.asideContent ? ('<aside class="notes" data-markdown>' + el.asideContent + '</aside>') : '';
 -      return '<script type="text/template">' + cnt + '</script>';
++        var content = el.content || el;
++        content += el.asideContent ? ('<aside class="notes" data-markdown>' + el.asideContent + '</aside>') : '';
++        return '<script type="text/template">' + content + '</script>';
      };
  
      var getForwardedAttributes = function(section) {
@@@ -59,15 -53,17 +61,18 @@@
          }
  
          return result.join( ' ' );
 -    }
 +    };
  
-     var slidifyMarkdown = function(markdown, separator, vertical, attributes) {
+     var slidifyMarkdown = function(markdown, separator, vertical, notes, attributes) {
  
          separator = separator || '^\n---\n$';
++        notes = notes || 'note:';
  
--        var reSeparator = new RegExp(separator + (vertical ? '|' + vertical : ''), 'mg'),
--            reHorSeparator = new RegExp(separator),
 -            notesSeparator = new RegExp(notes, 'mg'),
++        var separatorRegex = new RegExp( separator + ( vertical ? '|' + vertical : '' ), 'mg' ),
++            horizontalSeparatorRegex = new RegExp( separator ),
++            notesSeparatorRegex = new RegExp( notes, 'mgi' ),
              matches,
+             noteMatch,
              lastIndex = 0,
              isHorizontal,
              wasHorizontal = true,
@@@ -76,30 -74,41 +83,41 @@@
              markdownSections = '';
  
          // iterate until all blocks between separators are stacked up
--        while( matches = reSeparator.exec(markdown) ) {
++        while( matches = separatorRegex.exec( markdown ) ) {
+             asideContent = null;
  
              // determine direction (horizontal by default)
--            isHorizontal = reHorSeparator.test(matches[0]);
++            isHorizontal = horizontalSeparatorRegex.test( matches[0] );
  
              if( !isHorizontal && wasHorizontal ) {
                  // create vertical stack
--                sectionStack.push([]);
++                sectionStack.push( [] );
              }
  
              // pluck slide content from markdown input
--            content = markdown.substring(lastIndex, matches.index);
 -            noteMatch = content.split(notesSeparator);
++            content = markdown.substring( lastIndex, matches.index );
++            noteMatch = content.split( notesSeparatorRegex );
+ 
 -            if(noteMatch.length === 2) {
++            if( noteMatch.length === 2 ) {
+                 content = noteMatch[0];
+                 asideContent = noteMatch[1].trim();
+             }
+ 
+             slide = {
+                 content: content,
+                 asideContent: asideContent || ""
+             };
  
              if( isHorizontal && wasHorizontal ) {
                  // add to horizontal stack
-                 sectionStack.push(content);
+                 sectionStack.push(slide);
              } else {
                  // add to vertical stack
-                 sectionStack[sectionStack.length-1].push(content);
+                 sectionStack[sectionStack.length-1].push(slide);
              }
  
--            lastIndex = reSeparator.lastIndex;
++            lastIndex = separatorRegex.lastIndex;
              wasHorizontal = isHorizontal;
- 
          }
  
          // add the remaining slide
@@@ -107,15 -116,13 +125,13 @@@
  
          // flatten the hierarchical stack, and insert <section data-markdown> tags
          for( var k = 0, klen = sectionStack.length; k < klen; k++ ) {
-             // horizontal
-             if( typeof sectionStack[k] === 'string' ) {
-                 markdownSections += '<section '+ attributes +'>' +  twrap( sectionStack[k] )  + '</section>';
-             }
              // vertical
-             else {
 -           if(sectionStack[k].propertyIsEnumerable(length) && typeof sectionStack[k].splice === "function") {
++            if( sectionStack[k].propertyIsEnumerable(length) && typeof sectionStack[k].splice === 'function' ) {
                  markdownSections += '<section '+ attributes +'>' +
-                                         '<section>' +  sectionStack[k].map(twrap).join('</section><section>') + '</section>' +
+                                         '<section data-markdown>' +  sectionStack[k].map(twrap).join('</section><section data-markdown>') + '</section>' +
                                      '</section>';
+             } else {
+                 markdownSections += '<section '+ attributes +' data-markdown>' + twrap( sectionStack[k] ) + '</section>';
              }
          }