jekyll-responsive_image 1.0.0.pre3 → 1.0.0.pre4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: a46011bdb0ba186b7c73ad6955106be3916497d6
-  data.tar.gz: d27ed962eddedf75476e9c34b5913ef4bbdada59
+  metadata.gz: 52e6d96f085c965ad79ac57cf56ab3a40e32f21b
+  data.tar.gz: 26b46156c6e64631f955ea31f5f045ce5a95c03f
 SHA512:
-  metadata.gz: f0b4c556a2577f428db1230a1cf003bb768e5df38acef94de1a1449293f8675d1e793a46eecd11550f60874c0227fecd46e75e1402b1baa0d320d7ca877b745a
-  data.tar.gz: 5e494a47149dd30af3408861f28a044fc1cceddb12a3d0b40d4883bf5298b2c28203fde9c06353a9ac7df01bde780916a0d6bc67ef78257a1797ee4a73f94fb6
+  metadata.gz: 9e2617adb6db1e71a830c7e37a4721ab98f4c15815d192ee097737ebf78255ac1bb0fbb725550fbcd00dc78a7f7e67b668eb45053c0e92ba32b78131565349c7
+  data.tar.gz: ff8ea68c0724566e0adbe4dac3dc82c8dd7cdcd9e980e89f6c5bc32cf518b4d5899ef135db70ec861f122b79d6f8d9537e1f001063f084cc3fdd4afad0be792e

data/lib/jekyll/responsive_image.rb

@@ -5,12 +5,12 @@ require 'jekyll'
 require 'rmagick'
 
 require 'jekyll/responsive_image/version'
-require 'jekyll/responsive_image/defaults'
+require 'jekyll/responsive_image/config'
 require 'jekyll/responsive_image/utils'
 require 'jekyll/responsive_image/render_cache'
 require 'jekyll/responsive_image/image_processor'
 require 'jekyll/responsive_image/resize_handler'
-require 'jekyll/responsive_image/common'
+require 'jekyll/responsive_image/renderer'
 require 'jekyll/responsive_image/tag'
 require 'jekyll/responsive_image/block'
 require 'jekyll/responsive_image/extra_image_generator'

data/lib/jekyll/responsive_image/block.rb

@@ -1,20 +1,11 @@
-
-
-
-
-
-
-
-
-
 module Jekyll
-  class ResponsiveImage
+  module ResponsiveImage
     class Block < Liquid::Block
-      include Jekyll::ResponsiveImage::Common
+      include Jekyll::ResponsiveImage::Utils
 
       def render(context)
         attributes = YAML.load(super)
-        render_responsive_image(context, attributes)
+        Renderer.new(context.registers[:site], attributes).render_responsive_image
       end
     end
   end

data/lib/jekyll/responsive_image/config.rb

@@ -0,0 +1,22 @@
+module Jekyll
+  module ResponsiveImage
+    class Config
+      DEFAULTS = {
+        'default_quality'    => 85,
+        'base_path'          => 'assets',
+        'output_path_format' => 'assets/resized/%{filename}-%{width}x%{height}.%{extension}',
+        'sizes'              => [],
+        'extra_images'       => []
+      }
+
+      def initialize(site)
+        @site = site
+      end
+
+      def to_h
+        DEFAULTS.merge(@site.config['responsive_image'])
+                .merge(site_source: @site.source, site_dest: @site.dest)
+      end
+    end
+  end
+end

data/lib/jekyll/responsive_image/extra_image_generator.rb

@@ -1,16 +1,18 @@
 module Jekyll
-  class ResponsiveImage
+  module ResponsiveImage
     class ExtraImageGenerator < Jekyll::Generator
-      include Jekyll::ResponsiveImage::Common
+      include Jekyll::ResponsiveImage::Utils
 
       def generate(site)
-        config = make_config(site)
+        config = Config.new(site).to_h
+        site_source = Pathname.new(site.source)
 
         config['extra_images'].each do |pathspec|
           Dir.glob(site.in_source_dir(pathspec)) do |image_path|
-            relative_image_path = image_path.sub(/^#{Regexp.escape(image_path)}/, '')
+            path = Pathname.new(image_path)
+            relative_image_path = path.relative_path_from(site_source)
 
-            result = ImageProcessor.process(image_path, relative_image_path, config)
+            result = ImageProcessor.process(relative_image_path, config)
             result[:resized].each { |image| keep_resized_image!(site, image) }
           end
         end

data/lib/jekyll/responsive_image/image_processor.rb

@@ -1,22 +1,24 @@
 module Jekyll
-  class ResponsiveImage
+  module ResponsiveImage
     class ImageProcessor
       include ResponsiveImage::Utils
 
-      def process(absolute_image_path, relative_image_path, config)
-        raise SyntaxError.new("Invalid image path specified: #{absolute_image_path}") unless File.file?(absolute_image_path)
+      def process(image_path, config)
+        absolute_image_path = File.expand_path(image_path.to_s, config[:site_source])
+
+        raise SyntaxError.new("Invalid image path specified: #{image_path}") unless File.file?(absolute_image_path)
 
         resize_handler = ResizeHandler.new
         img = Magick::Image::read(absolute_image_path).first
 
         {
-          original: image_hash(config['base_path'], relative_image_path, img.columns, img.rows),
+          original: image_hash(config, image_path, img.columns, img.rows),
           resized: resize_handler.resize_image(img, config),
         }
       end
 
-      def self.process(absolute_image_path, relative_image_path, config)
-        self.new.process(absolute_image_path, relative_image_path, config)
+      def self.process(image_path, config)
+        self.new.process(image_path, config)
       end
     end
   end

data/lib/jekyll/responsive_image/render_cache.rb

@@ -1,5 +1,5 @@
 module Jekyll
-  class ResponsiveImage
+  module ResponsiveImage
     class RenderCache
       attr_accessor :store
 

data/lib/jekyll/responsive_image/renderer.rb

@@ -0,0 +1,37 @@
+module Jekyll
+  module ResponsiveImage
+    class Renderer
+      include Jekyll::ResponsiveImage::Utils
+
+      def initialize(site, attributes)
+        @site = site
+        @attributes = attributes
+      end
+
+      def render_responsive_image
+        cache_key = @attributes.to_s
+        result = @attributes['cache'] ? RenderCache.get(cache_key) : nil
+
+        if result.nil?
+          config = Config.new(@site).to_h
+
+          image = ImageProcessor.process(@attributes['path'], config)
+          @attributes['original'] = image[:original]
+          @attributes['resized'] = image[:resized]
+
+          @attributes['resized'].each { |resized| keep_resized_image!(@site, resized) }
+
+          image_template = @site.in_source_dir(@attributes['template'] || config['template'])
+          partial = File.read(image_template)
+          template = Liquid::Template.parse(partial)
+
+          result = template.render!(@attributes.merge(@site.site_payload))
+
+          RenderCache.set(cache_key, result)
+        end
+
+        result
+      end
+    end
+  end
+end

data/lib/jekyll/responsive_image/resize_handler.rb

@@ -1,5 +1,5 @@
 module Jekyll
-  class ResponsiveImage
+  module ResponsiveImage
     class ResizeHandler
       include ResponsiveImage::Utils
 
@@ -14,8 +14,8 @@ module Jekyll
           next unless needs_resizing?(img, width)
 
           image_path = img.filename.force_encoding(Encoding::UTF_8)
-          filepath = format_output_path(config['output_path_format'], config['base_path'], image_path, width, height)
-          resized.push(image_hash(config['base_path'], filepath, width, height))
+          filepath = format_output_path(config['output_path_format'], config, image_path, width, height)
+          resized.push(image_hash(config, filepath, width, height))
 
           site_source_filepath = File.expand_path(filepath, config[:site_source])
           site_dest_filepath = File.expand_path(filepath, config[:site_dest])
@@ -23,10 +23,10 @@ module Jekyll
           # Don't resize images more than once
           next if File.exist?(site_source_filepath)
 
-          ensure_output_dir_exists!(File.dirname(site_source_filepath))
-          ensure_output_dir_exists!(File.dirname(site_dest_filepath))
+          ensure_output_dir_exists!(site_source_filepath)
+          ensure_output_dir_exists!(site_dest_filepath)
 
-          Jekyll.logger.info "Generating #{filepath}"
+          Jekyll.logger.info "Generating #{site_source_filepath}"
 
           i = img.scale(ratio)
           i.write(site_source_filepath) do |f|
@@ -34,6 +34,7 @@ module Jekyll
           end
 
           # Ensure the generated file is copied to the _site directory
+          Jekyll.logger.info "Copying resized image to #{site_dest_filepath}"
           FileUtils.copy_file(site_source_filepath, site_dest_filepath)
 
           i.destroy!
@@ -44,11 +45,19 @@ module Jekyll
         resized
       end
 
+      def format_output_path(format, config, image_path, width, height)
+        params = symbolize_keys(image_hash(config, image_path, width, height))
+
+        Pathname.new(format % params).cleanpath.to_s
+      end
+
       def needs_resizing?(img, width)
         img.columns > width
       end
 
-      def ensure_output_dir_exists!(dir)
+      def ensure_output_dir_exists!(path)
+        dir = File.dirname(path)
+
         unless Dir.exist?(dir)
           Jekyll.logger.info "Creating output directory #{dir}"
           FileUtils.mkdir_p(dir)

data/lib/jekyll/responsive_image/tag.rb

@@ -1,8 +1,6 @@
 module Jekyll
-  class ResponsiveImage
+  module ResponsiveImage
     class Tag < Liquid::Tag
-      include Jekyll::ResponsiveImage::Common
-
       def initialize(tag_name, markup, tokens)
         super
 
@@ -15,7 +13,7 @@ module Jekyll
       end
 
       def render(context)
-        render_responsive_image(context, @attributes)
+        Renderer.new(context.registers[:site], @attributes).render_responsive_image
       end
     end
   end

data/lib/jekyll/responsive_image/utils.rb

@@ -1,8 +1,13 @@
 require 'pathname'
 
 module Jekyll
-  class ResponsiveImage
+  module ResponsiveImage
     module Utils
+      def keep_resized_image!(site, image)
+        keep_dir = File.dirname(image['path'])
+        site.config['keep_files'] << keep_dir unless site.config['keep_files'].include?(keep_dir)
+      end
+
       def symbolize_keys(hash)
         result = {}
         hash.each_key do |key|
@@ -11,17 +16,11 @@ module Jekyll
         result
       end
 
-      def format_output_path(format, base_path, image_path, width, height)
-        params = symbolize_keys(image_hash(base_path, image_path, width, height))
-
-        Pathname.new(format % params).cleanpath.to_s
-      end
-
       # Build a hash containing image information
-      def image_hash(base_path, image_path, width, height)
+      def image_hash(config, image_path, width, height)
         {
           'path'      => image_path,
-          'dirname'   => relative_dirname(base_path, image_path),
+          'dirname'   => relative_dirname(config, image_path),
           'basename'  => File.basename(image_path),
           'filename'  => File.basename(image_path, '.*'),
           'extension' => File.extname(image_path).delete('.'),
@@ -30,9 +29,9 @@ module Jekyll
         }
       end
 
-      def relative_dirname(base_path, image_path)
-        path = Pathname.new(image_path).expand_path
-        base = Pathname.new(base_path).expand_path
+      def relative_dirname(config, image_path)
+        path = Pathname.new(File.expand_path(image_path, config[:site_source]))
+        base = Pathname.new(File.expand_path(config['base_path'], config[:site_source]))
 
         path.relative_path_from(base).dirname.to_s.delete('.')
       end

data/lib/jekyll/responsive_image/version.rb

@@ -1,5 +1,5 @@
 module Jekyll
-  class ResponsiveImage
-    VERSION = '1.0.0.pre3'.freeze
+  module ResponsiveImage
+    VERSION = '1.0.0.pre4'.freeze
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: jekyll-responsive_image
 version: !ruby/object:Gem::Version
-  version: 1.0.0.pre3
+  version: 1.0.0.pre4
 platform: ruby
 authors:
 - Joseph Wynn
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-09-25 00:00:00.000000000 Z
+date: 2016-10-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: jekyll
@@ -61,11 +61,11 @@ extra_rdoc_files: []
 files:
 - lib/jekyll/responsive_image.rb
 - lib/jekyll/responsive_image/block.rb
-- lib/jekyll/responsive_image/common.rb
-- lib/jekyll/responsive_image/defaults.rb
+- lib/jekyll/responsive_image/config.rb
 - lib/jekyll/responsive_image/extra_image_generator.rb
 - lib/jekyll/responsive_image/image_processor.rb
 - lib/jekyll/responsive_image/render_cache.rb
+- lib/jekyll/responsive_image/renderer.rb
 - lib/jekyll/responsive_image/resize_handler.rb
 - lib/jekyll/responsive_image/tag.rb
 - lib/jekyll/responsive_image/utils.rb
@@ -90,7 +90,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: 1.3.1
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.5.1
+rubygems_version: 2.6.6
 signing_key: 
 specification_version: 4
 summary: Responsive images for Jekyll via srcset

data/lib/jekyll/responsive_image/common.rb

@@ -1,46 +0,0 @@
-module Jekyll
-  class ResponsiveImage
-    module Common
-      include Jekyll::ResponsiveImage::Utils
-
-      def make_config(site)
-        ResponsiveImage.defaults.dup
-                       .merge(site.config['responsive_image'])
-                       .merge(:site_source => site.source, :site_dest => site.dest)
-      end
-
-      def keep_resized_image!(site, image)
-        keep_dir = File.dirname(image['path'])
-        site.config['keep_files'] << keep_dir unless site.config['keep_files'].include?(keep_dir)
-      end
-
-      def render_responsive_image(context, attributes)
-        cache_key = attributes.to_s
-        result = attributes['cache'] ? RenderCache.get(cache_key) : nil
-
-        if result.nil?
-          site = context.registers[:site]
-          config = make_config(site)
-
-          absolute_image_path = site.in_source_dir(attributes['path'].to_s)
-          image = ImageProcessor.process(absolute_image_path, attributes['path'], config)
-          attributes['original'] = image[:original]
-          attributes['resized'] = image[:resized]
-
-          attributes['resized'].each { |resized| keep_resized_image!(site, resized) }
-
-          image_template = site.in_source_dir(attributes['template'] || config['template'])
-          partial = File.read(image_template)
-          template = Liquid::Template.parse(partial)
-
-          result = template.render!(attributes.merge(site.site_payload))
-
-
-          RenderCache.set(cache_key, result)
-        end
-
-        result
-      end
-    end
-  end
-end

data/lib/jekyll/responsive_image/defaults.rb

@@ -1,15 +0,0 @@
-module Jekyll
-  class ResponsiveImage
-    @defaults = {
-      'default_quality'    => 85,
-      'base_path'          => 'assets',
-      'output_path_format' => 'assets/resized/%{filename}-%{width}x%{height}.%{extension}',
-      'sizes'              => [],
-      'extra_images'       => []
-    }.freeze
-
-    class << self
-      attr_reader :defaults
-    end
-  end
-end