From e2121c2c2c8b941d08231078b2e9fb09612785ea Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?R=C3=A9gis=20Hanol?= Date: Wed, 4 Nov 2015 12:38:39 +0100 Subject: [PATCH] FIX: add 'max-width' to images with auto width/height in emails --- lib/email/styles.rb | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/lib/email/styles.rb b/lib/email/styles.rb index f518a6f999a..6f0dd6c2b2f 100644 --- a/lib/email/styles.rb +++ b/lib/email/styles.rb @@ -31,7 +31,6 @@ module Email # images @fragment.css('img').each do |img| - next if img['class'] == 'site-logo' if img['class'] == "emoji" || img['src'] =~ /plugins\/emoji/ @@ -56,9 +55,16 @@ module Email end end + # add max-width to big images + big_images = @fragment.css('img[width="auto"][height="auto"]') - + @fragment.css('aside.onebox img') - + @fragment.css('img.site-logo, img.emoji') + big_images.each do |img| + add_styles(img, 'max-width: 100%;') if img['style'] !~ /max-width/ + end + # attachments @fragment.css('a.attachment').each do |a| - # ensure all urls are absolute if a['href'] =~ /^\/[^\/]/ a['href'] = "#{Discourse.base_url}#{a['href']}"