WordPress.org
Get WordPress

Make WordPress Core

Opened 9 months ago

Closed 5 months ago

#63047 closed enhancement (fixed)

Add image upload/transform mappings to Site Health

Reported by: ironprogrammer's profile ironprogrammer Owned by: adamsilverstein's profile adamsilverstein
Milestone: 6.9 Priority: normal
Severity: normal Version:
Component: Site Health Keywords: has-patch commit
Focuses: Cc:

Description

For plugins and userland code that utilize the ​`image_editor_output_format` filter, it may not be clear to site operators what the final transforms are. This enhancement proposes adding the final upload->transform mappings information to the Site Health > Info tab, possibly under the existing "Media Handling" section. Displaying these mappings could prove useful, particularly when debugging final image output.

The motivation for this update comes from experiencing unexpected mappings, as reported in #62365.

Related: #62365.

Change History (10)

This ticket was mentioned in ​PR #8450 on ​WordPress/wordpress-develop by ​@yogeshbhutkar.
8 months ago #1

  • Keywords has-patch added

Trac ticket: https://core.trac.wordpress.org/ticket/63047

## Description

Adds visibility into image_editor_output_format filter transformations by displaying active format mappings in Site Health > Info > Media Handling section.

### Testing

Visit Site Health > Info > Media Handling section to verify the transform mappings display.

### Screenshots

_No Transform Applied_
https://github.com/user-attachments/assets/f0b4153e-673f-4a23-88ed-589024693c87

_Multiple Transforms Applied_
https://github.com/user-attachments/assets/5a3b13ff-ec33-45b7-80de-92201fd6a0e8

#2 @yogeshbhutkar
8 months ago

  • Keywords needs-testing added

#3 @adamsilverstein
8 months ago

  • Milestone changed from Awaiting Review to 6.9
  • Owner set to adamsilverstein
  • Status changed from new to assigned

#4 @adamsilverstein
8 months ago

  • Keywords needs-refresh added; needs-testing removed

@ironprogrammer looks good! I tested using playground and left some small feedback, otherwise seems like a nice addition - thanks!

#5 @ironprogrammer
8 months ago

Thanks for the PR, @yogeshbhutkar, and for reviewing it, @adamsilverstein! To be clear, this work was performed by @yogeshbhutkar, who gets the credit there 😉 Following Adam's suggestion, the update looks good from my perspective.

#6 @yogeshbhutkar
8 months ago

  • Keywords needs-refresh removed

Thanks, both! I've updated the patch to ensure consistent casing.

​@yogeshbhutkar commented on ​PR #8450:
6 months ago #7

Hi @adamsilverstein,

This PR has been approved for a while now but hasn't been merged or committed yet. Just checking in if we can move forward with getting it merged?

Thanks!

#8 @adamsilverstein
5 months ago

This PR has been approved for a while now but hasn't been merged or committed yet. Just checking in if we can move forward with getting it merged?

Yes @yogeshbhutkar - I will work on landing this.

#9 @adamsilverstein
5 months ago

  • Keywords commit added

#10 @adamsilverstein
5 months ago

  • Resolution set to fixed
  • Status changed from assigned to closed

In 60278:

Site Health: add image output mapping details to the info tab.

Add details to the info tab under media handling to show the mapping from uploaded image types to output types.

Props ironprogrammer, yogeshbhutkar.

Fixes #63047.

Note: See TracTickets for help on using tickets.