github-mirror/discourse
2
0
Fork 0
mirror of https://github.com/discourse/discourse.git synced 2025-01-19 04:52:45 +08:00
Code Issues Actions 1 Packages Projects Releases Wiki Activity

UX: only crops images taller than 18:9 instead of 16:9

Browse Source
This commit is contained in:
Régis Hanol 2018-01-31 22:31:16 +01:00
parent 4bcf9c5bf3
commit 7d2283167a
2 changed files with 5 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
lib/cooked_post_processor.rb
View File

@ -247,9 +247,9 @@ class CookedPostProcessor
rescue URI::InvalidURIError
end
# only crop when the image is taller than 16:9
# we only use 95% of that to allow for a small margin
MIN_RATIO_TO_CROP ||= (9.0 / 16.0) * 0.95
# only crop when the image is taller than 18:9
# we only use 90% of that to allow for a small margin
MIN_RATIO_TO_CROP ||= (9.0 / 18.0) * 0.9
def convert_to_link!(img)
src = img["src"]
@ -268,11 +268,7 @@ class CookedPostProcessor
return if original_width <= width && original_height <= height
return if original_width <= SiteSetting.max_image_width && original_height <= SiteSetting.max_image_height
crop = original_width.to_f / original_height.to_f < MIN_RATIO_TO_CROP
# prevent iPhone X screenshots from being cropped
crop &= original_width != 1125 && original_height != 2436
if crop
if crop = (original_width.to_f / original_height.to_f < MIN_RATIO_TO_CROP)
width, height = ImageSizer.crop(original_width, original_height)
img["width"] = width
img["height"] = height

2
spec/components/cooked_post_processor_spec.rb
View File

@ -180,7 +180,7 @@ describe CookedPostProcessor do
SiteSetting.create_thumbnails = true
Upload.expects(:get_from_url).returns(upload)
FastImage.expects(:size).returns([860, 1900])
FastImage.expects(:size).returns([860, 2000])
OptimizedImage.expects(:resize).never
OptimizedImage.expects(:crop).returns(true)