feat(AssetFieldType): implement version image optimization, remove version format conversion feature, add support for SVGs

Author: toastercup

app/cells/plugins/core/asset_info/show.haml

@@ -8,7 +8,7 @@
     - if asset_is_image?
       .mdl-card__title
         %h2.mdl-card__title-text
-          = image_tag(asset['versions']['original']['url'], style: 'max-height: 200px;')
+          = image_tag(asset['versions']['original']['url'], style: 'min-height: 50px; max-height: 200px;')
     .mdl-card__supporting-text
       %dl
         %dt Original Filename

app/uploaders/asset_uploader.rb

@@ -1,4 +1,5 @@
 require 'image_processing/mini_magick'
+require 'image_optim'
 
 class AssetUploader < Shrine
   include ImageProcessing::MiniMagick
@@ -25,14 +26,14 @@ class AssetUploader < Shrine
   end
 
   process(:store) do |io, context|
-    # TODO: Perform image optimizations (build plugin), support versions without processors or formatters
+    # TODO: support versions without processors
     context[:generated_hex] = SecureRandom.hex(8)
     versions = { original: io.download }
 
     if image?(io)
       versions.merge!(context[:config][:metadata][:versions].transform_values do |version|
         processed_version = send("#{version[:process][:method]}!", io.download, *version[:process][:config].values)
-        convert!(processed_version, version[:format])
+        optimize_image!(processed_version)
       end)
     end
 
@@ -52,4 +53,10 @@ def generate_location(io, context)
   def image?(io)
     MimeMagic.new(io.data['metadata']['mime_type']).mediatype == 'image'
   end
+
+  def optimize_image!(image)
+    image_optim = ImageOptim.new(pngout: false, svgo: false, allow_lossy: true) # TODO: implement per-Field/version image_optim configuration
+    pathname = image_optim.optimize_image!(image)
+    pathname ? pathname.open : image
+  end
 end

lib/tasks/cortex/core/media.rake

@@ -33,11 +33,12 @@ namespace :cortex do
                                title: fieldTitle.id
                              },
                              versions: {
-                               large: { process: { method: 'resize_to_limit', config: { width: '1800', height: '1800' } }, format: :jpg },
-                               medium: { process: { method: 'resize_to_limit', config: { width: '800', height: '800' } }, format: :jpg },
-                               default: { process: { method: 'resize_to_limit', config: { width: '300', height: '300' } }, format: :jpg },
-                               mini: { process: { method: 'resize_to_limit', config: { width: '100', height: '100' } }, format: :jpg },
-                               micro: { process: { method: 'resize_to_limit', config: { width: '50', height: '50' } }, format: :jpg },
+                               large: { process: { method: 'resize_to_limit', config: { width: '1800', height: '1800' } } },
+                               medium: { process: { method: 'resize_to_limit', config: { width: '800', height: '800' } } },
+                               default: { process: { method: 'resize_to_limit', config: { width: '300', height: '300' } } },
+                               mini: { process: { method: 'resize_to_limit', config: { width: '100', height: '100' } } },
+                               micro: { process: { method: 'resize_to_limit', config: { width: '50', height: '50' } } },
+                               rss: { process: { method: 'resize_to_limit', config: { width: '840', height: '840' } } },
                              },
                              keep_files: [:destroyed, :replaced],
                              path: 'media/<%= attachment %>/<%= original_name %>-<%= style %>-<%= generated_hex %>.<%= extension %>',