reverse_markdown 1.4.0 → 2.1.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 1945214c22af9c763471e6bf9375fe94f1618fd67af1de2a28a0c88cdc59c07a
-  data.tar.gz: 26352feda47de16934757ade063ec14d55a9a6c8364570a9b3b72c656deb803b
+  metadata.gz: cffa2a16f5ad4187cb23a2e6a1015121b943f1c83a688308cf0a480ec15d3c49
+  data.tar.gz: 6941ad7c07005ff4d5c115fd894ee2a0cf40a09367833268f4c55c018f30263a
 SHA512:
-  metadata.gz: 959cc8dff7496eec2b8828f6f059ababa4d008fe6b5aebc05d442b6c746b4d863b1f860f29c5620738273ed808a21b9a19182283d2c177e8b86d0553e53f01ad
-  data.tar.gz: 2580ead64498f94190f4e20cb3a475e1e8e5dc7f3343fe8d7ea4fcb4a7d9a163628facdb0eb00ad2b335edc3b9758dc83389c7dfd611e1325586d9a75e4e9a33
+  metadata.gz: 620ce03beae7238accdc6836e025dc7b19fe308a7bcbcd7143dc7472d09e7d25758a3f6c2080d5d75e60fee662bc87ab200f177b5b6f41fa5958bde4b569a49e
+  data.tar.gz: ea69f20a1e70e4f41d6024502af362351ccaf6ec8936b4dcadc2414202e994cdf8ddf08ebbd829c0722c373bbe7e87a223c3c913bf6d845be9308ef418f3be75

data/.travis.yml

@@ -1,23 +1,17 @@
-# before_install:
-#   - gem install bundler
+language: ruby
+cache: bundler
 
 rvm:
-  - 1.9.3
-  - 2.0.0
-  - 2.1.0
-  - 2.1.5
-  - 2.2.0
-  - 2.2.4
-  - 2.2.5
-  - 2.3.0
-  - 2.3.1
-  - 2.4.2
-  - 2.5.3
-  - 2.6.2
+  - 2.0
+  - 2.1
+  - 2.2
+  - 2.3
+  - 2.4
+  - 2.5
+  - 2.6
+  - 2.7
   - jruby-9.2.8.0
 
-script: "bundle exec rake spec"
-
 notifications:
   disabled: false
   recipients:

data/CHANGELOG.md

@@ -1,6 +1,17 @@
 # Change Log
 All notable changes to this project will be documented in this file.
 
+## 2.1.1 - October 2021
+- Fixes unintentional newline characters within lists with paragraphs, thanks @diogoosorio, see #93
+- Lets \n to be present in <pre> tag. solves #77 #78, thanks @shivabhusal
+
+## 2.1.0 - May 2020
+- Add support for `figure` tags, see #86, thanks @anshul78
+
+## 2.0.0 - March 2020
+- BREAKING: Dropped support for ruby 1.9.3
+- Add support for `details` and `summary` tags, see #85
+
 ## 1.4.0 – January 2020
 - BREAKING: jump links will no longer be ignored but treated as links, see #82
 

data/README.md

@@ -2,7 +2,7 @@
 
 Transform html into markdown. Useful for example if you want to import html into your markdown based application.
 
-[![Build Status](https://secure.travis-ci.org/xijo/reverse_markdown.png?branch=master)](https://travis-ci.org/xijo/reverse_markdown) [![Gem Version](https://badge.fury.io/rb/reverse_markdown.png)](http://badge.fury.io/rb/reverse_markdown) [![Code Climate](https://codeclimate.com/github/xijo/reverse_markdown.png)](https://codeclimate.com/github/xijo/reverse_markdown) [![Code Climate](https://codeclimate.com/github/xijo/reverse_markdown/coverage.png)](https://codeclimate.com/github/xijo/reverse_markdown)
+[![Build Status](https://secure.travis-ci.org/xijo/reverse_markdown.svg?branch=master)](https://travis-ci.org/xijo/reverse_markdown) [![Gem Version](https://badge.fury.io/rb/reverse_markdown.svg)](http://badge.fury.io/rb/reverse_markdown) [![Code Climate](https://codeclimate.com/github/xijo/reverse_markdown.svg)](https://codeclimate.com/github/xijo/reverse_markdown) [![Code Climate](https://codeclimate.com/github/xijo/reverse_markdown/coverage.png)](https://codeclimate.com/github/xijo/reverse_markdown)
 
 ## Changelog
 
@@ -11,7 +11,7 @@ See [Change Log](CHANGELOG.md)
 ## Requirements
 
 1. [Nokogiri](http://nokogiri.org/)
-2. Ruby 1.9.3 or higher
+2. Ruby 2.0.0 or higher
 
 ## Installation
 
@@ -29,7 +29,7 @@ gem 'reverse_markdown'
 
 ## Features
 
-- Supports all the established html tags like `h1`, `h2`, `h3`, `h4`, `h5`, `h6`, `p`, `em`, `strong`, `i`, `b`, `blockquote`, `code`, `img`, `a`, `hr`, `li`, `ol`, `ul`, `table`, `tr`, `th`, `td`, `br`
+- Supports all the established html tags like `h1`, `h2`, `h3`, `h4`, `h5`, `h6`, `p`, `em`, `strong`, `i`, `b`, `blockquote`, `code`, `img`, `a`, `hr`, `li`, `ol`, `ul`, `table`, `tr`, `th`, `td`, `br`, `figure`
 - Module based - if you miss a tag, just add it
 - Can deal with nested lists
 - Inline and block code is supported

data/lib/reverse_markdown/converters/details.rb

@@ -0,0 +1,25 @@
+module ReverseMarkdown
+    module Converters
+      class Details < Base
+        def convert(node, state = {})
+          content = treat_children(node, state.merge(already_processed: true))
+          if disabled? || content.strip.empty? || state[:already_processed]
+            content
+          else
+            "##{content}"
+          end
+        end
+
+        def enabled?
+          ReverseMarkdown.config.github_flavored
+        end
+
+        def disabled?
+          !enabled?
+        end
+      end
+
+      register :details, Details.new
+      register :summary, Details.new
+    end
+end

data/lib/reverse_markdown/converters/figcaption.rb

@@ -0,0 +1,15 @@
+module ReverseMarkdown
+  module Converters
+    class FigCaption < Base
+      def convert(node, state = {})
+        if node.text.strip.empty?
+          ""
+        else
+          "\n" << "_#{node.text.strip}_" << "\n"
+        end
+      end
+    end
+
+    register :figcaption, FigCaption.new
+  end
+end

data/lib/reverse_markdown/converters/figure.rb

@@ -0,0 +1,12 @@
+module ReverseMarkdown
+  module Converters
+    class Figure < Base
+      def convert(node, state = {})
+        content = treat_children(node, state)
+        "\n#{content.strip}\n"
+      end
+    end
+
+    register :figure, Figure.new
+  end
+end

data/lib/reverse_markdown/converters/li.rb

@@ -2,8 +2,8 @@ module ReverseMarkdown
   module Converters
     class Li < Base
       def convert(node, state = {})
-        contains_child_paragraph = node.children.first ? node.children.first.name == 'p' : false
-        content_node             = contains_child_paragraph ? node.children.first : node
+        contains_child_paragraph = node.first_element_child ? node.first_element_child.name == 'p' : false
+        content_node             = contains_child_paragraph ? node.first_element_child : node
         content                  = treat_children(content_node, state)
         indentation              = indentation_from(state)
         prefix                   = prefix_for(node)

data/lib/reverse_markdown/converters/pre.rb

@@ -15,8 +15,8 @@ module ReverseMarkdown
       # Override #treat as proposed in https://github.com/xijo/reverse_markdown/pull/69
       def treat(node, state)
         case node.name
-        when 'code'
-          node.text
+        when 'code', 'text'
+          node.text.strip
         when 'br'
           "\n"
         else

data/lib/reverse_markdown/version.rb

@@ -1,3 +1,3 @@
 module ReverseMarkdown
-  VERSION = '1.4.0'
+  VERSION = '2.1.1'
 end

data/lib/reverse_markdown.rb

@@ -13,7 +13,10 @@ require 'reverse_markdown/converters/code'
 require 'reverse_markdown/converters/del'
 require 'reverse_markdown/converters/div'
 require 'reverse_markdown/converters/drop'
+require 'reverse_markdown/converters/details'
 require 'reverse_markdown/converters/em'
+require 'reverse_markdown/converters/figcaption'
+require 'reverse_markdown/converters/figure'
 require 'reverse_markdown/converters/h'
 require 'reverse_markdown/converters/hr'
 require 'reverse_markdown/converters/ignore'

data/reverse_markdown.gemspec

@@ -23,6 +23,6 @@ Gem::Specification.new do |s|
   s.add_development_dependency 'simplecov'
   s.add_development_dependency 'rake'
   s.add_development_dependency 'kramdown'
-  # s.add_development_dependency 'byebug'
+  s.add_development_dependency 'byebug'
   s.add_development_dependency 'codeclimate-test-reporter'
 end

data/spec/assets/lists.html

@@ -44,6 +44,8 @@
       </li>
     </ul>
 
+    <ul><li> <p>I don't want to cleanup after the party!</p> </li></ul>
+
     <ul>
       <li>
         <p>li 1, p 1</p>

data/spec/components/lists_spec.rb

@@ -35,7 +35,8 @@ describe ReverseMarkdown do
   end
 
   context "lists containing embedded <p> tags" do
-    xit { is_expected.to match /\n- I want to have a party at my house!\n/ }
+    it { is_expected.to match /\n- I want to have a party at my house!\n/ }
+    it { is_expected.to match /\n- I don't want to cleanup after the party!\n/ }
   end
 
   context "list item containing multiple <p> tags" do

data/spec/components/paragraphs_spec.rb

@@ -1,7 +1,6 @@
 require 'spec_helper'
 
 describe ReverseMarkdown do
-
   let(:input)    { File.read('spec/assets/paragraphs.html') }
   let(:document) { Nokogiri::HTML(input) }
   subject { ReverseMarkdown.convert(input) }

data/spec/components/quotation_spec.rb

@@ -1,12 +1,10 @@
 require 'spec_helper'
 
 describe ReverseMarkdown do
-
   let(:input)    { File.read('spec/assets/quotation.html') }
   let(:document) { Nokogiri::HTML(input) }
   subject { ReverseMarkdown.convert(input) }
 
   it { is_expected.to match /^    Block of code$/ }
   it { is_expected.to include "\n> First quoted paragraph\n> \n> Second quoted paragraph" }
-
 end

data/spec/lib/reverse_markdown/converters/details_spec.rb

@@ -0,0 +1,23 @@
+require 'spec_helper'
+
+describe ReverseMarkdown::Converters::Details do
+  let(:converter) { ReverseMarkdown::Converters::Details.new }
+
+  context 'for standard markdown' do
+    before { ReverseMarkdown.config.github_flavored = false }
+
+    it 'handles details tags correctly' do
+      node = node_for("<details>foo</details>")
+      expect(converter.convert(node)).to include "foo"
+    end
+  end
+
+  context 'for github_flavored markdown' do
+    before { ReverseMarkdown.config.github_flavored = true }
+
+    it 'handles details tags correctly' do
+      node = node_for("<details>foo</details>")
+      expect(converter.convert(node)).to include "#foo"
+    end
+  end
+end

data/spec/lib/reverse_markdown/converters/figure_spec.rb

@@ -0,0 +1,17 @@
+require 'spec_helper'
+
+describe ReverseMarkdown::Converters::Figure do
+
+  let(:converter) { ReverseMarkdown::Converters::Figure.new }
+
+  it 'handles figure tags with figcaption correctly' do
+    node = node_for("<figure><img src='image.jpg' alt='img_alt'><figcaption>Figure Caption</figcaption></figure>")
+    expect(converter.convert(node)).to eq "\n![img_alt](image.jpg)\n_Figure Caption_\n"
+  end
+
+  it 'handles figure tags without figcaption correctly' do
+    node = node_for("<figure><img src='image.jpg' alt='img_alt'></figure>")
+    expect(converter.convert(node)).to eq "\n![img_alt](image.jpg)\n"
+  end
+
+end

data/spec/lib/reverse_markdown/converters/pre_spec.rb

@@ -12,10 +12,15 @@ describe ReverseMarkdown::Converters::Pre do
       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>")
@@ -41,6 +46,11 @@ describe ReverseMarkdown::Converters::Pre do
       expect(converter.convert(node)).to include "```\nfoo\nbar\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 "```\none\ntwo\nthree\nfour\n```"
+    end
+
     it 'handles code tags correctly' do
       node = node_for("<pre><code>foobar</code></pre>")
       expect(converter.convert(node)).to include "```\nfoobar\n```"

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: reverse_markdown
 version: !ruby/object:Gem::Version
-  version: 1.4.0
+  version: 2.1.1
 platform: ruby
 authors:
 - Johannes Opper
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-01-07 00:00:00.000000000 Z
+date: 2021-10-31 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: nokogiri
@@ -80,6 +80,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: byebug
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: codeclimate-test-reporter
   requirement: !ruby/object:Gem::Requirement
@@ -123,9 +137,12 @@ files:
 - lib/reverse_markdown/converters/bypass.rb
 - lib/reverse_markdown/converters/code.rb
 - lib/reverse_markdown/converters/del.rb
+- lib/reverse_markdown/converters/details.rb
 - lib/reverse_markdown/converters/div.rb
 - lib/reverse_markdown/converters/drop.rb
 - lib/reverse_markdown/converters/em.rb
+- lib/reverse_markdown/converters/figcaption.rb
+- lib/reverse_markdown/converters/figure.rb
 - lib/reverse_markdown/converters/h.rb
 - lib/reverse_markdown/converters/hr.rb
 - lib/reverse_markdown/converters/ignore.rb
@@ -173,6 +190,8 @@ files:
 - spec/lib/reverse_markdown/converters/blockquote_spec.rb
 - spec/lib/reverse_markdown/converters/br_spec.rb
 - spec/lib/reverse_markdown/converters/del_spec.rb
+- spec/lib/reverse_markdown/converters/details_spec.rb
+- spec/lib/reverse_markdown/converters/figure_spec.rb
 - spec/lib/reverse_markdown/converters/li_spec.rb
 - spec/lib/reverse_markdown/converters/pre_spec.rb
 - spec/lib/reverse_markdown/converters/strong_spec.rb
@@ -199,7 +218,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.3
+rubygems_version: 3.1.4
 signing_key: 
 specification_version: 4
 summary: Convert html code into markdown.
@@ -234,6 +253,8 @@ test_files:
 - spec/lib/reverse_markdown/converters/blockquote_spec.rb
 - spec/lib/reverse_markdown/converters/br_spec.rb
 - spec/lib/reverse_markdown/converters/del_spec.rb
+- spec/lib/reverse_markdown/converters/details_spec.rb
+- spec/lib/reverse_markdown/converters/figure_spec.rb
 - spec/lib/reverse_markdown/converters/li_spec.rb
 - spec/lib/reverse_markdown/converters/pre_spec.rb
 - spec/lib/reverse_markdown/converters/strong_spec.rb