kamifusen 1.9 → 1.10.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a82b418d1a07d8b33f845d6ded2bdf3d1309943e41350f8e003bbdcbfe867f08
-  data.tar.gz: 0d20f1c01167036520f1451f1d295d8c7a32d40e315b8ac6fdee707fb3236c43
+  metadata.gz: 56bd55356abc2f8c35701c288a89c82f0bde23ad807987eba2c129e28773e1ae
+  data.tar.gz: 10cc826b4de7b4de04de7ede89d3700c0fc9e9e581366df41f22d556ad56f788
 SHA512:
-  metadata.gz: 999758e2fece72df932546f261091b4ffc1584b85668fc9102852d703d4c45526a66ae980ae918f64a6884b99267d2c8c2d3c4646b78a7475ccfd5c01ccf64e0
-  data.tar.gz: be8d74901d1bb3f1859e01e6940c00aa2c9ee67aa086c34fb241de5e5adb510e14a93f39c040ba8dfa8708b1134d7637783775b3a7225cb400575d57643c316c
+  metadata.gz: 7a59010f32ab196d5228f0065eb7f46bdea8131cb1e861716346e25beb5e9458b574e9979fa63eb2469b8c36e8a69c6bf2f0742d8b365b04215bd9887aeb29be
+  data.tar.gz: 6ab4e8e652dca4ec6c6edd68a0c89c2301b1b9eb0e5396f322bdb26e535a2d358be4633aa29e501fdaabe95e739f9d842498807d2d03e247da801f3ef6c5b05d

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    kamifusen (1.8)
+    kamifusen (1.10.3)
       image_processing
       rails
 
@@ -76,7 +76,7 @@ GEM
     ffi (1.15.3)
     globalid (0.5.2)
       activesupport (>= 5.0)
-    i18n (1.8.10)
+    i18n (1.8.11)
       concurrent-ruby (~> 1.0)
     image_processing (1.12.1)
       mini_magick (>= 4.9.5, < 5)
@@ -89,20 +89,22 @@ GEM
       nokogiri (>= 1.5.9)
     mail (2.7.1)
       mini_mime (>= 0.1.1)
-    marcel (1.0.1)
+    marcel (1.0.2)
     method_source (1.0.0)
     mini_magick (4.11.0)
-    mini_mime (1.1.1)
+    mini_mime (1.1.2)
     mini_portile2 (2.6.1)
     minitest (5.14.4)
     nio4r (2.5.8)
-    nokogiri (1.12.4)
+    nokogiri (1.12.5)
       mini_portile2 (~> 2.6.1)
       racc (~> 1.4)
+    nokogiri (1.12.5-x86_64-darwin)
+      racc (~> 1.4)
     parallel (1.20.1)
     parser (3.0.1.1)
       ast (~> 2.4.1)
-    racc (1.5.2)
+    racc (1.6.0)
     rack (2.2.3)
     rack-test (1.1.0)
       rack (>= 1.0, < 3)
@@ -156,9 +158,9 @@ GEM
     sprockets (4.0.2)
       concurrent-ruby (~> 1.0)
       rack (> 1, < 3)
-    sprockets-rails (3.2.2)
-      actionpack (>= 4.0)
-      activesupport (>= 4.0)
+    sprockets-rails (3.4.0)
+      actionpack (>= 5.2)
+      activesupport (>= 5.2)
       sprockets (>= 3.0.0)
     sqlite3 (1.4.2)
     thor (1.1.0)
@@ -168,7 +170,7 @@ GEM
     websocket-driver (0.7.5)
       websocket-extensions (>= 0.1.0)
     websocket-extensions (0.1.5)
-    zeitwerk (2.4.2)
+    zeitwerk (2.5.1)
 
 PLATFORMS
   ruby
@@ -183,4 +185,4 @@ DEPENDENCIES
   sqlite3
 
 BUNDLED WITH
-   2.2.7
+   2.2.30

data/LICENSE

@@ -1,6 +1,6 @@
 MIT License
 
-Copyright (c) 2021 Sébastien Moulène
+Copyright (c) 2021 noesya
 
 Permission is hereby granted, free of charge, to any person obtaining a copy
 of this software and associated documentation files (the "Software"), to deal

data/README.md

@@ -32,6 +32,13 @@ If you want to disable webp, in config/initializers/kamifusen.rb:
 Kamifusen.with_webp = false
 ```
 
+If you render kamifusen tags inside a sanitized text, please make sure you allow these tags and attributes in `config/application.yml`:
+
+```ruby
+config.action_view.sanitized_allowed_tags = ['picture', 'source', 'img']
+config.action_view.sanitized_allowed_attributes = ['src', 'type', 'srcset', 'width', 'height', 'alt', 'sizes', 'loading', 'decoding']
+```
+
 ## Usage
 
 Simply use `kamifusen_tag` instead of `image_tag` in your rails views.## What's the problem?
@@ -103,7 +110,7 @@ Webp and AVIF are more efficient formats than jpg and png. They allow better com
 
 ### The final situation
 
-https://sebousan.github.io/kamifusen/
+https://noesya.github.io/kamifusen/
 
 The new helper is:
 

data/app/views/kamifusen/_view.html.erb

@@ -1,52 +1,48 @@
 <%
-# image_tag options
 options ||= {}
 alt = options[:alt]
-async = options.has_key?(:async)  ? options[:async]
-                                  : true
-active_storage_direct_url = options.has_key?(:active_storage_direct_url)  ? options[:active_storage_direct_url]
-                                                                          : false
+async = options.has_key?(:async) ? options[:async] : true
+active_storage_direct_url = options.has_key?(:active_storage_direct_url) ? options[:active_storage_direct_url] : false
 klass = options[:class]
 picture_class = options[:picture_class]
 width = options[:width]
 height = options[:height]
 
-sizes = Kamifusen.sizes
-quality = Kamifusen.quality
+variant_sizes = Kamifusen.sizes.dup
+quality = Kamifusen.quality.dup
 
-if  source&.metadata &&
-    source.metadata['analyzed'] &&
-    source.metadata.has_key?('width') &&
-    source.metadata.has_key?('height')
+sizes = options[:sizes] || {}
+sizes_value = sizes.map { |key, value| [key, value].join(' ').strip }.join(', ')
+
+if source&.analyzed? && source.metadata.has_key?('width') && source.metadata.has_key?('height')
   image_width = source.metadata['width']
   image_height = source.metadata['height']
   image_ratio = 1.0 * image_width / image_height
   if width.nil? && height.nil?
-    # Prendre width et height réelles de l'image
-    width = [image_width, sizes.last].min
-    height = width * image_ratio
+    # Image real dimensions
+    width = [image_width, variant_sizes.last].min
+    height = width / image_ratio
   elsif width.nil?
-    # Calculer la height sur ratio
+    # Calculated width, preserving the aspect ratio
     width = height * image_ratio
   elsif height.nil?
-    # Calcule la width sur ratio
+    # Calculated height, preserving the aspect ratio
     height = width / image_ratio
   else
-    # Width et height sont explicites,
-    # si le ratio demandé n'est pas celui de l'image,
-    # Redéfinir la height
+    # Explicit dimensions. We redefine the height if aspect ratio is not preserved.
     width = [image_width, width].min
     height = width / image_ratio
   end
   width = width.round
   height = height.round
 end
+
 parameters = ""
 parameters += " loading=\"lazy\" decoding=\"async\"" if async
-parameters += " alt=\"#{ alt }\"" if alt
-parameters += " width=\"#{ width }\"" if width
-parameters += " height=\"#{ height }\"" if height
-parameters += " class=\"#{ klass }\"" if klass
+parameters += " alt=\"#{alt}\"" if alt
+parameters += " width=\"#{width}\"" if width
+parameters += " height=\"#{height}\"" if height
+parameters += " class=\"#{klass}\"" if klass
 
 def kamifusen_process(variant, active_storage_direct_url)
   if active_storage_direct_url
@@ -54,45 +50,49 @@ def kamifusen_process(variant, active_storage_direct_url)
       # Pour générer la processed url, il faut savoir où sont stockées les images
       # https://discuss.rubyonrails.org/t/define-host-so-absolute-urls-work-in-development-and-test/75085
       # https://stackoverflow.com/questions/60425407/uriinvalidurierror-bad-uriis-not-uri-nil-active-storage-service-url
-      # Not compatible with Disk storage, will return nil
       url = variant.processed.url
     rescue
+      # Not compatible with DiskService, which returns a URI::InvalidURIError
     end
   end
-  url = url_for(variant) if url.nil?
+  url ||= url_for(variant)
   url
 end
 %>
 <% if source.variable? %>
-  <%#= "#{image_width} x #{image_height}, #{image_ratio} ratio => #{width} x #{height}<br>".html_safe %>
   <%
   # Computing
   if width
     width_retina = width * 2
-    sizes.reject! { |size| size > width_retina }
-    sizes << width_retina
-    sizes.uniq!
+    variant_sizes.reject! { |size| size > width_retina }
+    variant_sizes << width_retina
+    variant_sizes.uniq!
   end
-  default_width = sizes.max
+  default_width = variant_sizes.max
   default_width = width_retina if width_retina && width_retina > default_width
   if Kamifusen.with_webp
-    srcset_webp = sizes.map { |size|
+    srcset_webp = variant_sizes.map { |size|
       variant = source.variant(resize: "#{size}>", format: :webp, quality: quality)
-      "#{ kamifusen_process(variant, active_storage_direct_url) } #{ size }w"
+      "#{ kamifusen_process(variant, active_storage_direct_url) } #{size}w"
     }.join(', ')
   end
-  srcset_default = sizes.map { |size|
+  srcset_default = variant_sizes.map { |size|
     variant = source.variant(resize: "#{size}>", quality: quality)
-    "#{ kamifusen_process(variant, active_storage_direct_url) } #{ size }w"
+    "#{ kamifusen_process(variant, active_storage_direct_url) } #{size}w"
   }.join(', ')
   variant = source.variant(resize: "#{default_width}>", quality: quality)
   default = kamifusen_process(variant, active_storage_direct_url)
   %>
   <picture<%= " class=\"#{picture_class}\"".html_safe unless picture_class.blank? %>>
     <% if Kamifusen.with_webp %>
-      <source srcset="<%= srcset_webp %>" type="image/webp">
+      <source srcset="<%= srcset_webp %>"
+              <%= " sizes=\"#{sizes_value}\"".html_safe unless sizes.empty? %>
+              type="image/webp">
     <% end %>
-    <img src="<%= default %>" srcset="<%= srcset_default %>" type="<%= source.content_type %>"<%= raw parameters %>>
+    <source srcset="<%= srcset_default %>"
+            <%= " sizes=\"#{sizes_value}\"".html_safe unless sizes.empty? %>
+            type="<%= source.content_type %>">
+    <img src="<%= default %>" <%= raw parameters %>>
   </picture>
 <% else %>
   <picture>

data/lib/kamifusen/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Kamifusen
-  VERSION = "1.9"
+  VERSION = "1.10.3"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: kamifusen
 version: !ruby/object:Gem::Version
-  version: '1.9'
+  version: 1.10.3
 platform: ruby
 authors:
 - Sébastien Moulène
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2021-09-17 00:00:00.000000000 Z
+date: 2021-11-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -82,7 +82,6 @@ files:
 - Gemfile
 - Gemfile.lock
 - LICENSE
-- LICENSE.txt
 - README.md
 - Rakefile
 - app/views/kamifusen/_view.html.erb

data/LICENSE.txt

@@ -1,21 +0,0 @@
-The MIT License (MIT)
-
-Copyright (c) 2021 Arnaud Levy
-
-Permission is hereby granted, free of charge, to any person obtaining a copy
-of this software and associated documentation files (the "Software"), to deal
-in the Software without restriction, including without limitation the rights
-to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
-copies of the Software, and to permit persons to whom the Software is
-furnished to do so, subject to the following conditions:
-
-The above copyright notice and this permission notice shall be included in
-all copies or substantial portions of the Software.
-
-THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
-IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
-FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
-AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
-LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
-OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
-THE SOFTWARE.