fengyuanchen/compressorjs

Resize to non-proper scaling and checkmark effect

Open

#182 opened on Dec 5, 2023

View on GitHub
 (2 comments) (0 reactions) (0 assignees)JavaScript (4,810 stars) (473 forks)batch import
help wantedquestion

Description

Describe the bug Attached the 4K photo. 3D render has a textured layer to make it look more realistic. Resizing to 1920 or so works, however if you try to do some another ratio it fails miserably.

To Reproduce Use the playground to test. Upload photo, set maxWidth to 2048 etc.

Expected behavior No checkmark effet?

Screenshots Project-4K9 (after opening click again to zoom 100%)

Problem: Screenshot 2023-12-05 171231

Additional context Interpolation problem?

Contributor guide

Tech stack
javascript
Domain
frontend
Issue type
bug
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
3
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-3 hours
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
stale
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
needs investigation
Prerequisites
basic JavaScriptcanvas API
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
20
Research direction
Investigate the resize algorithm in compressorjs, focusing on interpolation settings. Check if the issue is related to incorrect use of canvas drawImage or custom resize logic. Review comments in issue #182 for user reproduction steps. Try to reproduce with different ratios and examine the code in the src directory for any scaling bugs.