Add some optimzation to avatar and header image variants

maxfierke · Nov 6, 2023 · 2c64f95 · 2c64f95
1 parent 76dcd8c
commit 2c64f95
Showing 1 changed file with 12 additions and 8 deletions.
diff --git a/app/models/user.rb b/app/models/user.rb
@@ -1,14 +1,14 @@
 class User < ActiveRecord::Base
   AVATAR_IMAGE_VARIANTS = {
-    small: { resize_to_fill: [100, 100] },
-    medium: { resize_to_fill: [256, 256] },
-    large: { resize_to_fill: [512, 512] },
+    small: { resize_to_fill: [100, 100], saver: { quality: 85 }, strip: true },
+    medium: { resize_to_fill: [256, 256], saver: { quality: 85 }, strip: true },
+    large: { resize_to_fill: [512, 512], saver: { quality: 85 }, strip: true },
   }.freeze
 
   HEADER_IMAGE_VARIANTS = {
-    small: { resize_to_fill: [800, 267], gravity: 'center' },
-    medium: { resize_to_fill: [1500, 500], gravity: 'center' },
-    large: { resize_to_fill: [3000, 1000], gravity: 'center' },
+    small: { resize_to_fill: [800, 267], gravity: 'center', saver: { quality: 85 }, strip: true },
+    medium: { resize_to_fill: [1500, 500], gravity: 'center', saver: { quality: 85 }, strip: true },
+    large: { resize_to_fill: [3000, 1000], gravity: 'center', saver: { quality: 85 }, strip: true },
   }.freeze
 
   self.ignored_columns = ["ga_view_id", "ga_property_id"].freeze
@@ -76,15 +76,19 @@ def gravatar_url(size = 256)
 
   def avatar_url(size = :medium)
     url = if avatar_image.attached?
-      Rails.application.routes.url_helpers.cdn_blob_url(avatar_image.variant(**AVATAR_IMAGE_VARIANTS[size]))
+      Rails.application.routes.url_helpers.cdn_blob_url(
+        avatar_image.variant(**AVATAR_IMAGE_VARIANTS[size])
+      )
     end
 
     url || gravatar_url(AVATAR_IMAGE_VARIANTS.dig(size, :resize_to_fill)&.first)
   end
 
   def header_image_url(size = :medium)
     if header_image.attached?
-      Rails.application.routes.url_helpers.cdn_blob_url(header_image.variant(**HEADER_IMAGE_VARIANTS[size]))
+      Rails.application.routes.url_helpers.cdn_blob_url(
+        header_image.variant(**HEADER_IMAGE_VARIANTS[size])
+      )
     end
   end