Linebreaks for blockquotes (#734)

* remove trailing newlines when parsing blockquotes

crates/wysiwyg/src/dom/parser/markdown/markdown_html_parser.rs

@@ -50,14 +50,18 @@ impl MarkdownHTMLParser {
             }
         };
 
-        // Format new lines.
+        // Remove any trailing newline characters from block tags
         let html = html
             .replace("<ul>\n", "<ul>")
             .replace("</ul>\n", "</ul>")
             .replace("<ol>\n", "<ol>")
             .replace("</ol>\n", "</ol>")
             .replace("</li>\n", "</li>")
-            .replace("<br />\n", "<br />");
+            .replace("<br />\n", "<br />")
+            .replace("<blockquote>\n", "<blockquote>")
+            .replace("</blockquote>\n", "</blockquote>")
+            .replace("<p>\n", "<p>")
+            .replace("</p>\n", "</p>");
 
         Ok(S::try_from(html).unwrap())
     }

crates/wysiwyg/src/tests/test_set_content.rs

@@ -74,3 +74,22 @@ fn set_contents_with_line_break_in_code_block() {
     let model = cm("<pre>\n<code>|Test</code></pre>");
     assert_eq!(tx(&model), "<pre><code>|Test</code></pre>");
 }
+
+#[test]
+fn set_content_from_markdown_blockquote() {
+    let mut model = cm("|");
+    model.set_content_from_markdown(&utf16("> quote")).unwrap();
+    assert_eq!(tx(&model), "<blockquote><p>quote|</p></blockquote>");
+}
+
+#[test]
+fn set_content_from_markdown_blockquote_multiline() {
+    let mut model = cm("|");
+    model
+        .set_content_from_markdown(&utf16("> quote\n\nfollowing text"))
+        .unwrap();
+    assert_eq!(
+        tx(&model),
+        "<blockquote><p>quote</p></blockquote><p>following text|</p>"
+    );
+}