Lets \n to be present in <pre> tag. solves xijo#77

shivabhusal · Oct 2, 2019 · c82a760 · c82a760
1 parent d18fd18
commit c82a760
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/lib/reverse_markdown/converters/text.rb b/lib/reverse_markdown/converters/text.rb
@@ -48,7 +48,7 @@ def remove_border_newlines(text)
       end
 
       def remove_inner_newlines(text)
-        text.tr("\r\n\t", ' ').squeeze(' ')
+        text.tr("\r\t", ' ').squeeze(' ')
       end
 
       def preserve_keychars_within_backticks(text)

diff --git a/spec/lib/reverse_markdown/converters/pre_spec.rb b/spec/lib/reverse_markdown/converters/pre_spec.rb
@@ -12,10 +12,15 @@
       expect(converter.convert(node)).to include "    puts foo\n"
     end
 
-    it 'preserves new lines' do
+    it 'preserves new lines as <br>' do
       node = node_for("<pre>one<br>two<br>three</pre>")
       expect(converter.convert(node)).to include "\n\n    one\n    two\n    three\n\n"
     end
+
+    it 'preserves new lines as <br> and \n' do
+      node = node_for("<pre>one\ntwo\nthree<br>four</pre>")
+      expect(converter.convert(node)).to include "\n\n    one\n    two\n    three\n    four\n\n"
+    end
 
     it 'handles code tags correctly' do
       node = node_for("<pre><code>foobar</code></pre>")