kamifusen 1.8 → 1.10.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 05b8beb4ef9751b96229c3b9d267a9c6fc99f34d4bb869228495ce51779b06e0
-  data.tar.gz: 879e686e9b394b0b34aecabaa35469ffc063c4c815fbd6b92fabc226c2fc6894
+  metadata.gz: 2fdd71ccadb407398ff0e93515e68ab5baf952c6126b21253f7eed7ed7a23ffe
+  data.tar.gz: bcf6bb686e133972a594c28992d2e4c44b8f481da07ebc48667871f1909f41a3
 SHA512:
-  metadata.gz: 9cf81c743cf040ed1beb800be796fbbdc9883d75efbea3ed17951598fcd32d2dbe095235fbe7d2d568a320de1a7a476d0889eea737dd1b3d7531fe667790d804
-  data.tar.gz: 0ccb61359dbe83cca6b6ae7a6c1a27fae0e0d52616d448582de9314deb8a1964e4fd5c1b23d11d4f3908cef7c0476813ab47dc1b8e2892b4ac9dcda8cf6259a9
+  metadata.gz: ea108d4ec53a1083f9c9f25ebcefeef09653f5929049cb6e7e7acb927e7e19fd83575967b2cd56531301e48758f287f748ffbc8dab9414251fd28b672c8009eb
+  data.tar.gz: 963dd8eef230ae2fe292a65ad3e74b01a2e1b47ebd264ca6dfbc3e6acda7a753af1acfa8faaf3e87c9dd90b426cc57d67589476dba2413fd12ce3fcf06831fa7

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    kamifusen (1.8)
+    kamifusen (1.10.2)
       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.3.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?

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

@@ -1,53 +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]
 
-# kamifusen settings
-sizes = [360, 375, 414, 576, 640, 750, 768, 828, 992, 1152, 1200, 1366, 1400, 1536, 1920, 1984, 2400]
-quality = 80
+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
+    # 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
@@ -55,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.8"
+  VERSION = "1.10.2"
 end

data/lib/kamifusen.rb

@@ -9,6 +9,36 @@ module Kamifusen
   mattr_accessor :with_webp
   @@with_webp = true
 
+  mattr_accessor :sizes
+  @@sizes = [
+    # 360, # Old android
+    375, # Old iPhone
+    # 414, # ?
+    576, # Tablets desktop
+    640, # iPhone SE, some tablets
+    750, # iPhone 6/7/8, 375@2x
+    768, # Old iPads, Old desktops
+    # 828, # ?
+    # 992, # Breakpoint bootstrap
+    1080, # iPhone 6/7/8 plus, 414@2.608 (sorry)
+    # 1125, # iPhone 10, 375@3x
+    1152,
+    # 1172, # iPhone 12, 390@3x
+    1200, # Desktop
+    1366, # Desktop
+    # 1400, # Breakpoint boostrap
+    1440, # Samsung Galaxy S20, 360@4x
+    1536, # Desktop, some iPads
+    1920, # Desktop 2k
+    2048, # Some iPad
+    2240, # Desktop iMac M1 chipset
+    2880, # Desktop MacBook Pro/Air 13" @2x
+    3072  # Desktop MacBook Pro 16" @2x
+  ]
+
+  mattr_accessor :quality
+  @@quality = 80
+
   class Engine < ::Rails::Engine
   end
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: kamifusen
 version: !ruby/object:Gem::Version
-  version: '1.8'
+  version: 1.10.2
 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-15 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.