wp-media / wp-media/imagify-plugin

Inconsistent behavior when resizing

Open
#486 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

type: enhancement
Dominant language
PHP
Stars
82
Forks
31
Avg merge
5d 10h
Merged PRs (30d)
9

Description

WordPress automatically resizes images on upload, Imagify doesn't need to do this anymore.
However, png images are not resized anymore, due to a recent change.

The good news: Imagify will resize it anyway.
The bad news: Imagify and WordPress don’t resize the same way. Imagify will test the image width only, while WordPress will test width and height. For example with a limit of 1000px and a 500x1500px png image, WP will resize it (the height is > 1000px) but Imagify won’t (the width is not > 1000px).

If we want to harmonize everything it will require to stop WP from resizing and let Imagify do the job all the time, the opposite of what we do so far.

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by reading the linked WordPress image.php logic and then locate Imagify’s upload resizing path. Compare how each path evaluates image width and height; done requires an agreed, consistent resizing behavior, but the issue does not specify which implementation should be changed.

Written by the indexing model from the issue text.

Assessment

Tech stack
php, wordpress
Domain
backend
Issue type
Bug
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.