jekyll-assets 0.9.1 → 0.9.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 5fdc398b27cedf84be860226e11d4d2de56f19f5
-  data.tar.gz: fe1671b80e6d5831844a48b2acd21c008fc49bf6
+  metadata.gz: be09a65858868010feaa42928c212ab8c02469fc
+  data.tar.gz: 2e86bb155a6c8f49775600a0230a4b795f2bd5cc
 SHA512:
-  metadata.gz: e83bb90c308e9497a5694c006275c44afb12991db55c69ad7783101f95f37d17bb060fc71b85723fbad4f7631884e41a91da3a8f34ef514f554f268753569acf
-  data.tar.gz: 67b4bc55cbb60365d02e4242daf6c0ea6008cf7c0998dc4201f8f9fd438802042e996786d59ceb76c279dc2bff970c1fa4de7e8f76cb585edcda2ad0d4e27146
+  metadata.gz: 588585739f1351fd9e22ce57ab6eefefffa2b5d35f16bd40614733fb25d0098fa15adc0e4913a33526c00b45d6c799113ea9b9a857fcbb0854a66d146ebc4212
+  data.tar.gz: 2f98a13683f3d5c6dcc3efdb9329f405d8ef44afbd404990183479a440ae05c441bf2e6b69b10c143a18f2d99136071200cca0705ab5a2210b79fcb2f7988e04

data/HISTORY.md

@@ -1,6 +1,10 @@
 ### master (unreleased)
 
 
+### 0.9.2 (2014-08-05)
+
+* Fix tags renderer with attributes. See #93. (Thanks @tomdiggle)
+
 
 ### 0.9.1 (2014-07-28)
 

data/lib/jekyll/assets_plugin/renderer.rb

@@ -12,6 +12,7 @@ module Jekyll
         @path   = logical_path.strip
 
         @path, _, @attrs = @path.partition(" ") if @path[" "]
+        @attrs.prepend(" ") if @attrs
       end
 
       def render_asset

data/lib/jekyll/assets_plugin/version.rb

@@ -1,5 +1,5 @@
 module Jekyll
   module AssetsPlugin
-    VERSION = "0.9.1"
+    VERSION = "0.9.2"
   end
 end

data/spec/lib/jekyll/assets_plugin/tag_spec.rb

@@ -12,14 +12,16 @@ RSpec.describe Jekyll::AssetsPlugin::Tag do
   end
 
   def render_tag(template, path, attrs = "")
-    template = Jekyll::AssetsPlugin::Renderer.const_get template
+    template  = Jekyll::AssetsPlugin::Renderer.const_get template
+    attrs     = " #{attrs}" unless attrs.empty?
+
     format template, :path => path, :attrs => attrs
   end
 
   context "{% image <file> %}" do
-    def tag_re(name)
+    def tag_re(name, attrs = "")
       file = "/assets/#{name}-[a-f0-9]{32}\.png"
-      Regexp.new "^#{render_tag :IMAGE, file}$"
+      Regexp.new "^#{render_tag :IMAGE, file, attrs}$"
     end
 
     context "when <file> exists" do
@@ -27,6 +29,11 @@ RSpec.describe Jekyll::AssetsPlugin::Tag do
       it { is_expected.to match tag_re("noise") }
     end
 
+    context "when <file> has an attribute" do
+      subject { render("{% image noise.png alt=\"Foobar\" %}") }
+      it { is_expected.to match tag_re("noise", "alt=\"Foobar\"") }
+    end
+
     context "when <file> does not exists" do
       subject { render("{% image not-found.png %}") }
       it { is_expected.to match not_found_error "not-found.png" }
@@ -34,9 +41,9 @@ RSpec.describe Jekyll::AssetsPlugin::Tag do
   end
 
   context "{% stylesheet <file> %}" do
-    def tag_re(name)
+    def tag_re(name, attrs = "")
       file = "/assets/#{name}-[a-f0-9]{32}\.css"
-      Regexp.new "^#{render_tag :STYLESHEET, file}$"
+      Regexp.new "^#{render_tag :STYLESHEET, file, attrs}$"
     end
 
     context "when <file> exists" do
@@ -44,6 +51,11 @@ RSpec.describe Jekyll::AssetsPlugin::Tag do
       it { is_expected.to match tag_re("app") }
     end
 
+    context "when <file> has an attribute" do
+      subject { render("{% stylesheet app.css type=\"text/css\" %}") }
+      it { is_expected.to match tag_re("app", "type=\"text/css\"") }
+    end
+
     context "when <file> name has multiple dots" do
       subject { render("{% stylesheet app.min %}") }
       it { is_expected.to match tag_re("app.min") }
@@ -61,9 +73,9 @@ RSpec.describe Jekyll::AssetsPlugin::Tag do
   end
 
   context "{% javascript <file> %}" do
-    def tag_re(name)
+    def tag_re(name, attrs = "")
       file = "/assets/#{name}-[a-f0-9]{32}\.js"
-      Regexp.new "^#{render_tag :JAVASCRIPT, file}$"
+      Regexp.new "^#{render_tag :JAVASCRIPT, file, attrs}$"
     end
 
     context "when <file> exists" do
@@ -71,6 +83,11 @@ RSpec.describe Jekyll::AssetsPlugin::Tag do
       it { is_expected.to match tag_re("app") }
     end
 
+    context "when <file> has an attribute" do
+      subject { render("{% javascript app.js type=\"text/javascript\" %}") }
+      it { is_expected.to match tag_re("app", "type=\"text/javascript\"") }
+    end
+
     context "when <file> name has multiple dots" do
       subject { render("{% javascript app.min %}") }
       it { is_expected.to match tag_re("app.min") }

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: jekyll-assets
 version: !ruby/object:Gem::Version
-  version: 0.9.1
+  version: 0.9.2
 platform: ruby
 authors:
 - Aleksey V Zapparov
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-07-28 00:00:00.000000000 Z
+date: 2014-08-05 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: jekyll
@@ -205,7 +205,7 @@ rubyforge_project:
 rubygems_version: 2.2.2
 signing_key: 
 specification_version: 4
-summary: jekyll-assets-0.9.1
+summary: jekyll-assets-0.9.2
 test_files:
 - spec/fixtures/.gitignore
 - spec/fixtures/_assets/alert.js