[wp-trac] [WordPress Trac] #50523: Redesign the admin Image Editor

WordPress Trac noreply at wordpress.org
Sun Jan 1 00:22:32 UTC 2023


#50523: Redesign the admin Image Editor
-------------------------------------------------+-------------------------
 Reporter:  afercia                              |       Owner:  joedolson
     Type:  enhancement                          |      Status:  accepted
 Priority:  normal                               |   Milestone:  6.2
Component:  Media                                |     Version:
 Severity:  normal                               |  Resolution:
 Keywords:  needs-design early needs-testing     |     Focuses:  ui,
  has-testing-info                               |  accessibility
-------------------------------------------------+-------------------------
Changes (by bgoewert):

 * keywords:  needs-design early needs-testing => needs-design early needs-
     testing has-testing-info


Comment:

 == Test Report

 Patch tested:
 [https://core.trac.wordpress.org/attachment/ticket/50523/50523.5.diff
 50523.5.diff]

 === Steps to Test
 1. Upload image ([https://source.unsplash.com/random/1280x720?busy random
 image])
 1. Select image
 1. Edit more details
 1. Edit image
 1. Crop
 1. Undo
 1. Scale
 1. Rotate
 1. Flip
 1. Save
 1. Restore

 === Expected Results

 All image actions work
 - Crop
 - Undo
 - Scale
 - Rotate
 - Flip
 - Save
 - Restore

 === Environment
 * OS: Pop!_OS 22.04
 * Web Server: Docker Desktop
 * PHP: 7.4.33
 * WordPress: 6.2-alpha-54642-src
 * Browser: Firefox 108, Chrome 108
 * Theme: Twenty Twenty-Three
 * Active Plugins: None

 === Actual Results

 - ✅ Crop
 - ✅ Undo
 - ✅ Scale
 - ✅ Rotate
 - ✅ Flip
 - ✅ Save
 - ✅ Restore

 === Additional Notes
 - Are the crop/scale action settings supposed to be in a sidebar? Because
 they were not.
 - Noticed that scale saves automatically and there is no way to undo a
 scale unless you restore the original image.
 - Also noticed the scale ignores any additional unsaved changes and
 applies without those changes.
 - After applying a crop, the "Crop Image" settings do not get removed
 unless you click crop again.


 === Supplemental Artifacts
 Didn't embed images so as to not create a long comment.

 ''Before applying patch''
 https://i.imgur.com/7TR9LNS.png

 ''After applying patch''
 https://i.imgur.com/pUZCf9H.jpg

 ''Cropping''
 https://i.imgur.com/Wi7b7A5.png

 ''After applying crop''
 https://i.imgur.com/1NEa0Jy.png

 ''Scaling''
 https://i.imgur.com/6OoBdny.png

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/50523#comment:46>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list