jekyll-toc 0.7.0.beta1 → 0.7.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 2b7578981ba66427ed37a31d9705ad0e174d5646d31f5cd028fbe28038249a83
-  data.tar.gz: 63cfa7ca781467175dea8072975b41416fc556e2e4fa3c0f77f741ff0746c401
+  metadata.gz: bf842397d274621c986cf5182baaed15cff4f406e309741621ca897545525cf0
+  data.tar.gz: ac1181c4a77d63773b6fa12d3b646582b01e77aba41104291cf8c2bff3a6babb
 SHA512:
-  metadata.gz: 639bb631c1370b4c0f01f5769fa79048f883603d9582441d2c7d214e89bd2e578d6c174b72c48d5d1acaf52e496fe59332c8b6626eb4b1786c1ee6934bcce542
-  data.tar.gz: 120b7f1bafb23b34c99149d4db56418644e9918b436a9753ecf07ac5cabb19a3410eac999785277cc38bf6e3a88de2bc84efbac4e3162485250d460affcc5d92
+  metadata.gz: f01826275203ce38ebdc8d8ce427e91bf9f8e1c4e437803caf5a3073fe44e5cdc4b022f4319199ec3eeca2b834cf8a282bfd38843bfb5d7bd918958c90994722
+  data.tar.gz: 0df513610cb1aa29706f069404424d66d0e50252b3df93254c8419cf7a297497b96eb9ba33506b53f3f56681950f4f14881fb3156e2f50713d708ee4fe4d8e9e

data/README.md

@@ -5,6 +5,16 @@
 [![Code Climate](https://codeclimate.com/github/toshimaru/jekyll-toc/badges/gpa.svg)](https://codeclimate.com/github/toshimaru/jekyll-toc)
 [![Test Coverage](https://api.codeclimate.com/v1/badges/cd56b207f327603662a1/test_coverage)](https://codeclimate.com/github/toshimaru/jekyll-toc/test_coverage)
 
+# Table of Contents
+
+- [Installation](#installation)
+- [Usage](#usage)
+- [Generated HTML](#generated-html)
+- [Customization](#customization)
+  - [Skip TOC](#skip-toc)
+  - [TOC levels](#toc-levels)
+  - [CSS Styling](#css-styling)
+
 # Installation
 
 Add jekyll-toc plugin in your site's `Gemfile`, and run `bundle install`.
@@ -73,6 +83,8 @@ location with the `toc_only` filter.
 
 ## Generated HTML
 
+![screenshot](https://user-images.githubusercontent.com/803398/28401295-0dcfb7ca-6d54-11e7-892b-2f2e6ca755a7.png)
+
 jekyll-toc generates an unordered list. The HTML output is as follows.
 
 ```html
@@ -97,13 +109,9 @@ jekyll-toc generates an unordered list. The HTML output is as follows.
 </ul>
 ```
 
-It looks like the image below.
-
-![screenshot](https://user-images.githubusercontent.com/803398/28401295-0dcfb7ca-6d54-11e7-892b-2f2e6ca755a7.png)
-
 ## Customization
 
-### Skip TOC(`no_toc`)
+### Skip TOC
 
 The heding is ignored in the toc when you add `no_toc` to the class.
 
@@ -113,7 +121,7 @@ The heding is ignored in the toc when you add `no_toc` to the class.
 <h2>h2</h2>
 ```
 
-### TOC level
+### TOC levels
 
 The toc levels can be configured on `_config.yml`.
 

data/lib/table_of_contents/parser.rb

@@ -19,7 +19,7 @@ module Jekyll
       end
 
       def build_toc
-        %(<ul class="section-nav">\n#{build_toc_list(@entries, last_ul_used: true)}</ul>)
+        %(<ul class="section-nav">\n#{build_toc_list(@entries)}</ul>)
       end
 
       def inject_anchors_into_html
@@ -68,7 +68,7 @@ module Jekyll
       end
 
       # Returns the list items for entries
-      def build_toc_list(entries, last_ul_used: false)
+      def build_toc_list(entries)
         i = 0
         toc_list = ''.dup
         min_h_num = entries.map { |e| e[:h_num] }.min
@@ -83,7 +83,7 @@ module Jekyll
               next_entry = entries[i + 1]
               if next_entry[:h_num] > min_h_num
                 nest_entries = get_nest_entries(entries[i + 1, entries.count], min_h_num)
-                toc_list << %(\n<ul>\n#{build_toc_list(nest_entries, last_ul_used: true)}</ul>\n)
+                toc_list << %(\n<ul>\n#{build_toc_list(nest_entries)}</ul>\n)
                 i += nest_entries.count
               end
             end
@@ -92,11 +92,7 @@ module Jekyll
           elsif entry[:h_num] > min_h_num
             # If the current entry should be indented in the list, generate a sublist
             nest_entries = get_nest_entries(entries[i, entries.count], min_h_num)
-            if last_ul_used
-              toc_list << build_toc_list(nest_entries, last_ul_used: true)
-            else
-              toc_list << %(<ul>\n#{build_toc_list(nest_entries, last_ul_used: true)}</ul>\n)
-            end
+            toc_list << build_toc_list(nest_entries)
             i += nest_entries.count - 1
           end
           i += 1
@@ -116,7 +112,7 @@ module Jekyll
       end
 
       def toc_headings
-        @toc_levels.map { |level| "h#{level}" }.join(",")
+        @toc_levels.map { |level| "h#{level}" }.join(',')
       end
 
       def generate_option_hash(options)

data/lib/version.rb

@@ -1,3 +1,3 @@
 module JekyllToc
-  VERSION = '0.7.0.beta1'.freeze
+  VERSION = '0.7.0'.freeze
 end

data/test/test_kramdown_list.rb

@@ -4,7 +4,6 @@ require 'test_helper'
 
 class TestKramdownList < Minitest::Test
   # NOTE: kramdown automatically injects `id` attribute
-  # TODO: test Japanese heading
   def test_kramdown_heading
     text = <<-MARKDOWN
 # h1
@@ -14,10 +13,27 @@ class TestKramdownList < Minitest::Test
     expected = <<-HTML
 <h1 id="h1">h1</h1>
 
-<h2 id=\"h2\">h2</h2>
+<h2 id="h2">h2</h2>
     HTML
+    actual = Kramdown::Document.new(text).to_html
 
-    assert_equal(expected, Kramdown::Document.new(text).to_html)
+    assert_equal(expected, actual)
+  end
+
+  def test_japanese_heading
+    text = <<-MARKDOWN
+# 日本語見出し１
+
+## 日本語見出し２
+    MARKDOWN
+    expected = <<-HTML
+<h1 id="section">日本語見出し１</h1>
+
+<h2 id="section-1">日本語見出し２</h2>
+    HTML
+    actual = Kramdown::Document.new(text).to_html
+
+    assert_equal(expected, actual)
   end
 
   def test_kramdown_list_1
@@ -49,8 +65,9 @@ class TestKramdownList < Minitest::Test
   </li>
 </ul>
     HTML
+    actual = Kramdown::Document.new(text).to_html
 
-    assert_equal(expected, Kramdown::Document.new(text).to_html)
+    assert_equal(expected, actual)
   end
 
   def test_kramdown_list_2
@@ -79,8 +96,9 @@ class TestKramdownList < Minitest::Test
   </li>
 </ul>
     HTML
+    actual = Kramdown::Document.new(text).to_html
 
-    assert_equal(expected, Kramdown::Document.new(text).to_html)
+    assert_equal(expected, actual)
   end
 
   def test_kramdown_list_3
@@ -95,8 +113,9 @@ class TestKramdownList < Minitest::Test
 * level-3   * level-2 * level-1
 </code></pre>
     HTML
+    actual = Kramdown::Document.new(text).to_html
 
-    assert_equal(expected, Kramdown::Document.new(text).to_html)
+    assert_equal(expected, actual)
   end
 
   def test_kramdown_list_4
@@ -119,18 +138,19 @@ class TestKramdownList < Minitest::Test
   <li>level-1</li>
 </ul>
     HTML
+    actual = Kramdown::Document.new(text).to_html
 
-    assert_equal(expected, Kramdown::Document.new(text).to_html)
+    assert_equal(expected, actual)
   end
 
-    def test_kramdown_list_5
-      text = <<-MARKDOWN
+  def test_kramdown_list_5
+    text = <<-MARKDOWN
 * level-1
     * level-3
   * level-2
 * level-1
-      MARKDOWN
-      expected = <<-HTML
+    MARKDOWN
+    expected = <<-HTML
 <ul>
   <li>level-1
     <ul>
@@ -140,8 +160,9 @@ class TestKramdownList < Minitest::Test
   </li>
   <li>level-1</li>
 </ul>
-      HTML
+    HTML
+    actual = Kramdown::Document.new(text).to_html
 
-      assert_equal(expected, Kramdown::Document.new(text).to_html)
-    end
+    assert_equal(expected, actual)
+  end
 end

data/test/test_option_error.rb

@@ -3,7 +3,7 @@
 require 'test_helper'
 
 class TestOptionError < Minitest::Test
-  BASE_HTML = "<h1>h1</h1>"
+  BASE_HTML = '<h1>h1</h1>'
   EXPECTED_HTML = <<-HTML
 <ul class="section-nav">
 <li class="toc-entry toc-h1"><a href="#h1">h1</a></li>
@@ -18,14 +18,14 @@ class TestOptionError < Minitest::Test
   end
 
   def test_option_is_epmty_string
-    parser = Jekyll::TableOfContents::Parser.new(BASE_HTML, "")
+    parser = Jekyll::TableOfContents::Parser.new(BASE_HTML, '')
     doc = Nokogiri::HTML(parser.toc)
     expected = EXPECTED_HTML
     assert_equal(expected, doc.css('ul.section-nav').to_s)
   end
 
   def test_option_is_string
-    parser = Jekyll::TableOfContents::Parser.new(BASE_HTML, "string")
+    parser = Jekyll::TableOfContents::Parser.new(BASE_HTML, 'string')
     doc = Nokogiri::HTML(parser.toc)
     expected = EXPECTED_HTML
     assert_equal(expected, doc.css('ul.section-nav').to_s)

data/test/test_various_toc_html.rb

@@ -44,6 +44,12 @@ class TestVariousTocHtml < Minitest::Test
 <h4 class="no_toc">no_toc h4</h4>
   HTML
 
+  TEST_JAPANESE_HTML = <<-HTML
+<h1>あ</h1>
+<h2>い</h2>
+<h3>う</h3>
+  HTML
+
   def test_nested_toc
     parser = Jekyll::TableOfContents::Parser.new(TEST_HTML_1)
     doc = Nokogiri::HTML(parser.toc)
@@ -177,4 +183,27 @@ class TestVariousTocHtml < Minitest::Test
 
     assert_equal(expected, actual)
   end
+
+  def test_japanese_toc
+    parser = Jekyll::TableOfContents::Parser.new(TEST_JAPANESE_HTML)
+    doc = Nokogiri::HTML(parser.toc)
+    expected = <<-HTML
+<ul class="section-nav">
+<li class="toc-entry toc-h1">
+<a href="#%E3%81%82">あ</a>
+<ul>
+<li class="toc-entry toc-h2">
+<a href="#%E3%81%84">い</a>
+<ul>
+<li class="toc-entry toc-h3"><a href="#%E3%81%86">う</a></li>
+</ul>
+</li>
+</ul>
+</li>
+</ul>
+    HTML
+    actual = doc.css('ul.section-nav').to_s
+
+    assert_equal(expected, actual)
+  end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: jekyll-toc
 version: !ruby/object:Gem::Version
-  version: 0.7.0.beta1
+  version: 0.7.0
 platform: ruby
 authors:
 - toshimaru
@@ -153,9 +153,9 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: 2.2.2
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ">"
+  - - ">="
     - !ruby/object:Gem::Version
-      version: 1.3.1
+      version: '0'
 requirements: []
 rubyforge_project: 
 rubygems_version: 2.7.6