summaryrefslogtreecommitdiffstats
path: root/app/file.go
diff options
context:
space:
mode:
authorHarrison Healey <harrisonmhealey@gmail.com>2017-07-05 17:01:32 -0400
committerSaturnino Abril <saturnino.abril@gmail.com>2017-07-06 05:01:32 +0800
commit29b22f13c59de76f1e0485e707aa5a61034798d4 (patch)
tree73e9c31a5dbc70c86f521413d609d1560ec94384 /app/file.go
parent4508fa09e22dc8ea2694786a5282f3aa1fa805e8 (diff)
downloadchat-29b22f13c59de76f1e0485e707aa5a61034798d4.tar.gz
chat-29b22f13c59de76f1e0485e707aa5a61034798d4.tar.bz2
chat-29b22f13c59de76f1e0485e707aa5a61034798d4.zip
PLT-6975 Fixed image thumbnail size being calculated incorrectly (#6855)
Diffstat (limited to 'app/file.go')
-rw-r--r--app/file.go13
1 files changed, 9 insertions, 4 deletions
diff --git a/app/file.go b/app/file.go
index 9786ba961..d21fd4a14 100644
--- a/app/file.go
+++ b/app/file.go
@@ -55,7 +55,7 @@ const (
MaxImageSize = 6048 * 4032 // 24 megapixels, roughly 36MB as a raw image
IMAGE_THUMBNAIL_PIXEL_WIDTH = 120
- IMAGE_THUMBNAIL_PIXEL_HEIGTH = 100
+ IMAGE_THUMBNAIL_PIXEL_HEIGHT = 100
IMAGE_PREVIEW_PIXEL_WIDTH = 1024
)
@@ -556,11 +556,16 @@ func getImageOrientation(input io.Reader) (int, error) {
}
func generateThumbnailImage(img image.Image, thumbnailPath string, width int, height int) {
+ thumbWidth := float64(IMAGE_THUMBNAIL_PIXEL_WIDTH)
+ thumbHeight := float64(IMAGE_THUMBNAIL_PIXEL_HEIGHT)
+ imgWidth := float64(width)
+ imgHeight := float64(height)
+
var thumbnail image.Image
- if height < IMAGE_THUMBNAIL_PIXEL_HEIGTH && width < IMAGE_THUMBNAIL_PIXEL_WIDTH {
+ if imgHeight < IMAGE_THUMBNAIL_PIXEL_HEIGHT && imgWidth < thumbWidth {
thumbnail = img
- } else if height/width < IMAGE_THUMBNAIL_PIXEL_HEIGTH/IMAGE_THUMBNAIL_PIXEL_WIDTH {
- thumbnail = imaging.Resize(img, 0, IMAGE_THUMBNAIL_PIXEL_HEIGTH, imaging.Lanczos)
+ } else if imgHeight/imgWidth < thumbHeight/thumbWidth {
+ thumbnail = imaging.Resize(img, 0, IMAGE_THUMBNAIL_PIXEL_HEIGHT, imaging.Lanczos)
} else {
thumbnail = imaging.Resize(img, IMAGE_THUMBNAIL_PIXEL_WIDTH, 0, imaging.Lanczos)
}