jekyll_picture_tag 1.14.0 → 2.0.0pre1

data/lib/jekyll_picture_tag/defaults/global.rb

@@ -0,0 +1,18 @@
+module PictureTag
+  # Default settings for _config.yml
+  DEFAULT_CONFIG = {
+    'picture' => {
+      'source' => '',
+      'output' => 'generated',
+      'suppress_warnings' => false,
+      'relative_url' => true,
+      'cdn_environments' => ['production'],
+      'nomarkdown' => true,
+      'ignore_missing_images' => false,
+      'disabled' => false,
+      'fast_build' => false,
+      'ignore_baseurl' => false,
+      'baseurl_key' => 'baseurl'
+    }
+  }.freeze
+end

data/lib/jekyll_picture_tag/defaults/presets.rb

@@ -0,0 +1,57 @@
+module PictureTag
+  DEFAULT_PRESET = { 'markup' => 'auto',
+                     'formats' => ['original'],
+                     'widths' => [400, 600, 800, 1000],
+                     'fallback_width' => 800,
+                     'fallback_format' => 'original',
+                     'noscript' => false,
+                     'link_source' => false,
+                     'quality' => 75,
+                     'format_quality' => { 'webp' => 50,
+                                           'avif' => 30,
+                                           'jp2' => 30 },
+                     'data_sizes' => true,
+                     'keep' => 'attention',
+                     'dimension_attributes' => false,
+                     'strip_metadata' => true,
+                     'image_options' => {
+                       'avif' => { 'compression' => 'av1', 'speed' => 8 }
+                     } }.freeze
+
+  STOCK_PRESETS = {
+    'jpt-webp' => { 'formats' => %w[webp original] },
+
+    'jpt-avif' => { 'formats' => %w[avif webp original] },
+
+    'jpt-lazy' => { 'markup' => 'data_auto',
+                    'noscript' => true,
+                    'formats' => %w[webp original],
+                    'attributes' => { 'parent' => 'class="lazy"' } },
+
+    'jpt-loaded' => { 'formats' => %w[avif jp2 webp original],
+                      'dimension_attributes' => true },
+
+    'jpt-direct' => { 'markup' => 'direct_url',
+                      'fallback_format' => 'webp',
+                      'fallback_width' => 600 },
+
+    'jpt-thumbnail' => { 'base_width' => 250,
+                         'pixel_ratios' => [1, 1.5, 2],
+                         'formats' => %w[webp original],
+                         'fallback_width' => 250,
+                         'attributes' => { 'picture' => 'class="icon"' } },
+
+    'jpt-avatar' => { 'base_width' => 100,
+                      'pixel_ratios' => [1, 1.5, 2],
+                      'fallback_width' => 100,
+                      'crop' => '1:1' }
+  }.freeze
+
+  STOCK_MEDIA_QUERIES = {
+    'jpt-mobile' => 'max-width: 480px',
+    'jpt-tablet' => 'max-width: 768',
+    'jpt-laptop' => 'max-width: 1024px',
+    'jpt-desktop' => 'max-width: 1200',
+    'jpt-wide' => 'min-width: 1201'
+  }.freeze
+end

data/lib/jekyll_picture_tag/images.rb

@@ -1,3 +1,4 @@
+require_relative 'images/image_file'
 require_relative 'images/generated_image'
 require_relative 'images/img_uri'
 require_relative 'images/source_image'

data/lib/jekyll_picture_tag/images/generated_image.rb

@@ -1,18 +1,20 @@
-require 'mini_magick'
+require 'ruby-vips'
 
 module PictureTag
-  # Represents a generated image file.
+  # Represents a generated image, but not the file itself. Its purpose is to
+  # make its properties available for query, and hand them off to the ImageFile
+  # class for generation.
   class GeneratedImage
-    attr_reader :width, :format
+    attr_reader :width
 
-    include MiniMagick
-
-    def initialize(source_file:, width:, format:, crop: nil, gravity: '')
+    def initialize(source_file:, width:, format:)
       @source = source_file
       @width  = width
-      @format = process_format format
-      @crop = crop
-      @gravity = gravity
+      @raw_format = format
+    end
+
+    def format
+      @format ||= process_format(@raw_format)
     end
 
     def exists?
@@ -43,80 +45,40 @@ module PictureTag
 
     # Post crop
     def source_width
-      update_cache unless cache[:width]
-
-      cache[:width]
+      image.width
     end
 
     # Post crop
     def source_height
-      update_cache unless cache[:height]
-
-      cache[:height]
+      image.height
     end
 
-    private
-
-    # We exclude width and format from the cache name, since it isn't specific to them.
-    def cache
-      @cache ||= Cache::Generated.new("#{@source.base_name}-#{id}")
+    def quality
+      PictureTag.quality(format, width)
     end
 
-    def update_cache
-      return if @source.missing
-
-      # Ensure it's generated:
-      image
-
-      cache[:width] = @source_dimensions[:width]
-      cache[:height] = @source_dimensions[:height]
-
-      cache.write
-    end
+    private
 
-    # Hash all inputs and truncate, so we know when they change without getting too long.
+    # Hash all inputs and truncate, so we know when they change without getting
+    # too long.
     # /home/dave/my_blog/_site/generated/somefolder/myimage-100-1234abcde.jpg
     #                                                           ^^^^^^^^^
     def id
-      @id ||= Digest::MD5.hexdigest([@source.digest, @crop, @gravity, quality].join)[0..8]
+      @id ||= Digest::MD5.hexdigest(settings.join)[0..8]
     end
 
-    def image
-      return @image if defined? @image
-
-      # Post crop, before resizing and reformatting
-      @source_dimensions = { width: image_base.width, height: image_base.height }
-
-      @image = image_base
+    def settings
+      [@source.digest, @source.crop, @source.keep, quality]
     end
 
-    def image_base
-      @image_base ||= Image.open(@source.name).combine_options do |i|
-        if PictureTag.preset['strip_metadata']
-          i.auto_orient
-          i.strip
-        end
-
-        if @crop
-          i.gravity @gravity
-          i.crop @crop
-        end
-      end
+    def image
+      @image ||= Vips::Image.new_from_file @source.name
     end
 
     def generate_image
-      puts 'Generating new image file: ' + name
-
-      image.format(@format, 0, { resize: "#{@width}x", quality: quality })
-      FileUtils.mkdir_p(File.dirname(absolute_filename))
-
-      image.write absolute_filename
-
-      FileUtils.chmod(0o644, absolute_filename)
-    end
+      return if @source.missing
 
-    def quality
-      PictureTag.quality(@format, @width)
+      ImageFile.new(@source, self)
     end
 
     def process_format(format)

data/lib/jekyll_picture_tag/images/image_file.rb

@@ -0,0 +1,90 @@
+module PictureTag
+  # Basically a wrapper class for vips. Handles image operations.
+  # Vips returns new images for Crop, resize, and autorotate operations.
+  # Quality, metadata stripping, and format are applied on write.
+  class ImageFile
+    def initialize(source, base)
+      @source = source
+      @base = base
+
+      build
+    end
+
+    private
+
+    attr_reader :source, :base
+
+    def build
+      notify
+
+      mkdir
+
+      image = load_image
+
+      image = process(image)
+
+      write(image)
+    end
+
+    # Processing pipeline
+    def process(image)
+      image = crop(image) if source.crop?
+
+      image = resize(image)
+
+      image.autorot
+    end
+
+    def write_opts
+      opts = PictureTag.preset['image_options'][@base.format] || {}
+
+      opts[:strip] = PictureTag.preset['strip_metadata']
+
+      # gifs don't accept a quality setting.
+      opts[:Q] = base.quality unless base.format == 'gif'
+
+      opts.transform_keys(&:to_sym)
+    end
+
+    def load_image
+      Vips::Image.new_from_file source.name
+    end
+
+    def write(image)
+      begin
+        image.write_to_file(base.absolute_filename, **write_opts)
+      rescue Vips::Error
+        # If vips can't handle it, fall back to imagemagick.
+        opts = write_opts.transform_keys do |key|
+          key == :Q ? :quality : key
+        end
+
+        image.magicksave(base.absolute_filename, **opts)
+      end
+
+      # Fix permissions. TODO - still necessary?
+      FileUtils.chmod(0o644, base.absolute_filename)
+    end
+
+    def notify
+      puts 'Generating new image file: ' + base.name
+    end
+
+    def resize(image)
+      image.resize(scale_value)
+    end
+
+    def crop(image)
+      image.smartcrop(*source.dimensions,
+                      interesting: PictureTag.keep(@source.media_preset))
+    end
+
+    def scale_value
+      base.width.to_f / source.width
+    end
+
+    def mkdir
+      FileUtils.mkdir_p(File.dirname(base.absolute_filename))
+    end
+  end
+end

data/lib/jekyll_picture_tag/images/img_uri.rb

@@ -18,7 +18,7 @@ module PictureTag
     # |     domain       |  baseurl   |       directory       | filename
     def to_s
       Addressable::URI.escape(
-        File.join(domain, baseurl, directory, @filename)
+        File.join(domain, PictureTag.baseurl, directory, @filename)
       )
     end
 
@@ -29,23 +29,14 @@ module PictureTag
     # |     domain       |  baseurl   |    j-p-t output dir   | filename
     def domain
       if PictureTag.cdn?
-        PictureTag.pconfig['cdn_url']
-      elsif PictureTag.pconfig['relative_url']
+        PictureTag.cdn_url
+      elsif PictureTag.relative_url
         ''
       else
         PictureTag.config['url'] || ''
       end
     end
 
-    # https://example.com/my-base-path/assets/generated-images/image.jpg
-    #                     ^^^^^^^^^^^^^
-    # |     domain       |  baseurl   |       directory       | filename
-    def baseurl
-      return '' if PictureTag.pconfig['ignore_baseurl']
-
-      PictureTag.config[PictureTag.pconfig['baseurl_key']] || ''
-    end
-
     # https://example.com/my-base-path/assets/generated-images/image.jpg
     #                                  ^^^^^^^^^^^^^^^^^^^^^^^^
     # |     domain       |  baseurl   |       directory       | filename

data/lib/jekyll_picture_tag/images/source_image.rb

@@ -5,7 +5,7 @@ module PictureTag
   class SourceImage
     attr_reader :shortname, :missing, :media_preset
 
-    include MiniMagick
+    # include MiniMagick
 
     def initialize(relative_filename, media_preset = nil)
       # /home/dave/my_blog/assets/images/somefolder/myimage.jpg
@@ -21,12 +21,38 @@ module PictureTag
       @digest ||= cache[:digest] || ''
     end
 
+    def crop
+      PictureTag.crop(media_preset)
+    end
+
+    def crop?
+      !crop.nil?
+    end
+
+    def keep
+      PictureTag.keep(media_preset)
+    end
+
+    def dimensions
+      [width, height]
+    end
+
     def width
-      @width ||= cache[:width] || 999_999
+      return raw_width unless crop?
+
+      [raw_width, (raw_height * cropped_aspect)].min.round
     end
 
     def height
-      @height ||= cache[:height] || 999_999
+      return raw_height unless crop?
+
+      [raw_height, (raw_width / cropped_aspect)].min.round
+    end
+
+    def cropped_aspect
+      return Utils.aspect_float(raw_width, raw_height) unless crop?
+
+      Utils.aspect_float(*crop.split(':').map(&:to_f))
     end
 
     # /home/dave/my_blog/assets/images/somefolder/myimage.jpg
@@ -44,13 +70,23 @@ module PictureTag
     # /home/dave/my_blog/assets/images/somefolder/myimage.jpg
     #                                                     ^^^
     def ext
-      @ext ||= File.extname(name)[1..-1].downcase
+      @ext ||= File.extname(name)[1..].downcase
     end
 
     private
 
+    # pre-crop
+    def raw_width
+      @raw_width ||= @missing ? 999_999 : image.width
+    end
+
+    # pre-crop
+    def raw_height
+      @raw_height ||= @missing ? 999_999 : image.height
+    end
+
     def cache
-      @cache ||= Cache::Source.new(@shortname)
+      @cache ||= Cache.new(@shortname)
     end
 
     def missing?
@@ -75,14 +111,12 @@ module PictureTag
 
     def update_cache
       cache[:digest] = source_digest
-      cache[:width] = image.width
-      cache[:height] = image.height
 
       cache.write
     end
 
     def image
-      @image ||= Image.open(name)
+      @image ||= Vips::Image.new_from_file(name)
     end
 
     def source_digest
@@ -90,7 +124,8 @@ module PictureTag
     end
 
     def missing_image_warning
-      "JPT Could not find #{name}. Your site will have broken images. Continuing."
+      "JPT Could not find #{name}. " \
+        'Your site will have broken images. Continuing.'
     end
 
     def missing_image_error

data/lib/jekyll_picture_tag/instructions.rb

@@ -1,6 +1,70 @@
-require_relative './instructions/set'
-require_relative './instructions/configuration'
-require_relative './instructions/html_attributes'
-require_relative './instructions/preset'
-require_relative './instructions/tag_parser'
-require_relative './instructions/arg_splitter'
+module PictureTag
+  # Instructions obtain, validate, and typecast/coerce input values. These
+  # inputs are either taken directly from jekyll's inputs, or handled by parsers
+  # first.
+  #
+  # Logic which affects only a single setting belongs in Instructions, while
+  # logic which affects multiple settings belongs in Parsers.
+  #
+  # Since instruction classes are so small, we define several per file in the
+  # instructions directory to save on boilerplate. All fall under the
+  # Instructions module namespace.
+  module Instructions
+    # Generic instruction, meant to be inherited. Children of this class must
+    # override the source method, and likely want to override valid?, coerce,
+    # and error_message as applicable.
+    class Instruction
+      # Memoized value of the given instruction. This is the public API.
+      def value
+        return @value if defined?(@value)
+
+        raise ArgumentError, error_message unless valid?
+
+        @value = coerced
+      end
+
+      private
+
+      # Source(s) of truth - where does this setting come from? Logic does not
+      # belong here. If information comes from muliple places, return an array
+      # or a hash.
+      def source
+        raise NotImplementedError
+      end
+
+      # Determine whether or not the input(s) are valid.
+      def valid?
+        true
+      end
+
+      # Convert input(s) to output.
+      def coerce
+        source
+      end
+
+      # Message returned if validation fails. Override this with something more
+      # helpful.
+      def error_message
+        "JPT - #{setting_name} received an invalid argument: #{source}"
+      end
+
+      def coerced
+        return @coerced if defined?(@coerced)
+
+        @coerced = coerce
+      end
+
+      def setting_name
+        Utils.snakeize(self.class.to_s.split('::').last)
+      end
+    end
+  end
+end
+
+# Load Parents
+Dir[File.dirname(__FILE__) + '/instructions/parents/*.rb']
+  .sort.each { |file| require file }
+
+# Load children:
+Dir[File.dirname(__FILE__) + '/instructions/children/*.rb']
+  .sort.each { |file| require file }