[wp-trac] [WordPress Trac] #42063: Media: Indicate if image is used as a site option

WordPress Trac noreply at wordpress.org
Mon Aug 31 12:56:42 UTC 2020


#42063: Media: Indicate if image is used as a site option
---------------------------------------+---------------------------------
 Reporter:  melchoyce                  |       Owner:  (none)
     Type:  enhancement                |      Status:  new
 Priority:  normal                     |   Milestone:  Future Release
Component:  Media                      |     Version:
 Severity:  normal                     |  Resolution:
 Keywords:  has-patch has-screenshots  |     Focuses:  ui, administration
---------------------------------------+---------------------------------
Changes (by Mista-Flo):

 * keywords:  needs-patch => has-patch has-screenshots


Comment:

 All right, so I was working on this ticket. A bit more complicated than I
 thought, here's why:

 - List view display cropped images as well
 - If I upload a new image, and crop it, and use it as the logo for
 example, the media state will be displayed along the cropped image
 attachment in list view. But in grid view, the cropped images are not
 displayed, meaning I won't know which image is used as a logo. If I don't
 crop the original image, I will see the media state (with my patch
 applied).

 So I will need some feedback about this inconsistency.

 However I have uploaded a patch that:
 - Create a new get_media_states in wp-admin/includes/template.php, it just
 basically mimic the same function as get_post_states, meaning I could use
 it without printing the same HTML output.
 - It will display the media state as "Current state" in the image detail
 modal, feel free to give feedback about the wording here.

 I'll upload two screenshots. One with the current behaviour in the list
 mode, another with the output of the patch.

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


More information about the wp-trac mailing list