[wp-trac] [WordPress Trac] #48304: Big Image: Revisit 'threshold' suffix on file names

[WordPress Trac]

#48304: Big Image: Revisit 'threshold' suffix on file names
--------------------------+-----------------------------------
 Reporter:  kraftbj       |      Owner:  (none)
     Type:  defect (bug)  |     Status:  new
 Priority:  normal        |  Milestone:  5.3
Component:  Media         |    Version:  trunk
 Severity:  normal        |   Keywords:  2nd-opinion has-patch
  Focuses:                |
--------------------------+-----------------------------------
 After #46076, when uploading a big image (over 2560px by default), there
 is special processing to create a new "full" size image that acts as the
 largest size that should usually be used by WordPress.

 The naming convention employed by 46076 for this reduced image is the
 value of the threshold alone, e.g. `image-2560.png` for a file of
 `image.png` uploaded.

 WordPress, previously, used both the height and width for all created
 sizes, e.g. `image-2560x2000.png`.

 Whenever a plugin/service is looking for the original image, it can
 commonly look to strip the suffix. For this new size, the suffix style has
 changed resulting in cases where it may not be able to realize it is not
 the original image.

 Question: Are we open to not having a special naming convention for the
 threshold-limited file and let WordPress give it the standard `image-
 2560x2000.jpg` file name?

 Attaching a patch if we want to use the more traditional naming scheme and
 marking as 5.3 to ensure review before the new functionality ships as
 stable.

 cc: @azaozz

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/48304>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform