Ability to remove a previously-used Header Image from the list of rotating "random" header images

[TomAuger]

Being able to add a bunch of header images and then selecting the "Random" option in the Appearance > Header section is awesome! However, it is equally important to be able to remove one/some/all of those header images from the list of rotating images. The current UI does not support this.

An approach that would be the least disruptive to the current UI layout would be to enable a "remove from list" link (I wouldn't use the word "delete" because it's not deleting the media resource) when the user hovers over the image. The link could appear below the thumbnail, much as the Post's Quick Edit link appear under the list of Posts/Pages.

[ocean90]

Yes, please.

[tomauger]

I should take a look at this, now that I know a thing or two about a thing or two. I don't know about getting it in in time for 3.9 though.

[tomauger]

First scratch at removing header images

[tomauger]

This initial patch is a quick fix, allowing header images to be removed from the "available images" pool. It has a small jQuery error right now that prevents the user from removing multiple images without a refresh, which I will address in the next update.

The whole header image approach is bogus though and should be completely refactored (see #27202) once we get the Media Modal settled. Currently selecting ANY media library item as a Header Image will automatically create a new attachment, even if it's exactly cropped. So if you remove an image from the list, the next time you add it, your library will get yet another duplicate of that image.

[SergeyBiryukov]

Replying to tomauger:

Currently selecting ANY media library item as a Header Image will automatically create a new attachment, even if it's exactly cropped. So if you remove an image from the list, the next time you add it, your library will get yet another duplicate of that image.

Related: [20806], #20657.

As noted in IRC, it was designed that way to prevent orphaning header image if the original attachment is deleted (and vice versa).

Currently you can remove an image from the list of available header images by deleting that attachment from Media Library. So I guess the patch should just allow to do that on the Custom Header screen itself.

[ocean90]

It's now possible in the Customizer to remove an image from the list. It uses the custom-header-remove ajax action, see Custom_Image_Header::ajax_header_remove(). This should be reused.

[celloexpressions]

As of [30459], very few users will be dealing with header images outside of the Customizer. We could still do this for those users (mainly no-js), but I don't think it's worth the effort.

If we do something here, #18959 is related and could possibly end up being a duplicate.

[wonderboymusic]

wontfix since [30459]