From 93884e2e93571a7e6996b542b03717784b4793d0 Mon Sep 17 00:00:00 2001 From: Aaron van Geffen Date: Tue, 24 Nov 2020 11:33:16 +0100 Subject: [PATCH] Fix initial slicing dimensions in CropEditor.setDefaultCrop. Subtle bug. This has been in for years... :) --- public/js/crop_editor.js | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/public/js/crop_editor.js b/public/js/crop_editor.js index ddfab11..e1e89e5 100644 --- a/public/js/crop_editor.js +++ b/public/js/crop_editor.js @@ -94,7 +94,7 @@ class CropEditor { // Cropping from the centre? if (cropMethod === "c") { // Crop vertically from the centre, using the entire width. - if (sourceAspectRatio < cropAspectRatio) { + if (sourceAspectRatio <= cropAspectRatio) { this.crop_width.value = source.naturalWidth; this.crop_height.value = Math.ceil(source.naturalWidth / cropAspectRatio); this.source_x.value = 0; @@ -111,9 +111,9 @@ class CropEditor { // Cropping a top or bottom slice? else { // Can we actually take a top or bottom slice from the original image? - if (sourceAspectRatio < cropAspectRatio) { + if (sourceAspectRatio <= cropAspectRatio) { this.crop_width.value = source.naturalWidth; - this.crop_height.value = Math.floor(source.naturalHeight / cropAspectRatio); + this.crop_height.value = Math.floor(source.naturalWidth / cropAspectRatio); this.source_x.value = "0"; this.source_y.value = cropMethod.indexOf("t") !== -1 ? "0" : source.naturalHeight - this.crop_height.value; }