Merge pull request #10 from jywarren/newline-issue

double newline fix

Author: jywarren

dist/inlineMarkdownEditor.js

@@ -10238,7 +10238,9 @@ inlineMarkdownEditor = function inlineMarkdownEditor(o) {
   var el = $(o.selector);
   o.originalMarkdown = el.html();
   // split by double-newline:
-  var sections = o.originalMarkdown.split('\n\n');
+  var sections = o.originalMarkdown
+                  .replace(/\n[\n]+/g, "\n\n")
+                  .split('\n\n');
   var editableSections = [];
   // we also do this inside processSection, but independently track here:
   sections.forEach(function forEachSection(section, index) {

package.json

@@ -1,6 +1,6 @@
 {
   "name": "inline-markdown-editor",
-  "version": "0.2.0",
+  "version": "0.2.1",
   "description": "An inline wysiwyg markdown document editor based on replacing string subsections. WYSIWYG possible via woofmark.",
   "main": "dist/inlineMarkdownEditor.js",
   "scripts": {

spec/javascripts/editor_spec.js

@@ -32,12 +32,23 @@ describe("Editor", function() {
   });
 
   it("generates the right number of sections", function() {
-    expect(editor.sections.length).toBe(6);
-    expect($('.inline-section').length).toBe(6);
+    expect(editor.sections.length).toBe(8);
+    expect($('.inline-section').length).toBe(8);
     expect(editor.sections[0]).toBe('');
-    expect(editor.editableSections.length).toBe(3);
-    expect($('.inline-edit-form').length).toBe(3);
-    expect($('.inline-edit-form textarea').length).toBe(3);
+    expect(editor.editableSections.length).toBe(5);
+    expect($('.inline-edit-form').length).toBe(5);
+    expect($('.inline-edit-form textarea').length).toBe(5);
+  });
+
+  it("correctly splits up sections", function() {
+    expect(editor.sections[0]).toBe("");
+    expect(editor.sections[1]).toBe("### Hello, World");
+    expect(editor.sections[2]).toBe("This is a paragraph.");
+    expect(editor.sections[3]).toBe("* this\n* is\n* a\n* list");
+    expect(editor.sections[4]).toBe("****");
+    expect(editor.sections[5]).toBe("### Hello again, World");
+    expect(editor.sections[6]).toBe("### Hello finally");
+    expect(editor.sections[7]).toBe("");
   });
 
 });

spec/javascripts/fixtures/index.html

@@ -11,4 +11,11 @@
 
 ****
 
+
+### Hello again, World
+
+
+
+### Hello finally
+
 </div>

src/inlineMarkdownEditor.js

@@ -9,7 +9,9 @@ inlineMarkdownEditor = function inlineMarkdownEditor(o) {
   var el = $(o.selector);
   o.originalMarkdown = el.html();
   // split by double-newline:
-  var sections = o.originalMarkdown.split('\n\n');
+  var sections = o.originalMarkdown
+                  .replace(/\n[\n]+/g, "\n\n")
+                  .split('\n\n');
   var editableSections = [];
   // we also do this inside processSection, but independently track here:
   sections.forEach(function forEachSection(section, index) {