Make WordPress Core

Opened 2 years ago

Last modified 2 years ago

#22150 new defect (bug)

Customizer: Remove Image doesn't remove from Media Library

Reported by: pento Owned by:
Milestone: Awaiting Review Priority: normal
Severity: minor Version: 3.4
Component: Upload Keywords:
Focuses: Cc:


After uploading an image using the Theme Customizer, a "Remove Image" link appears - this seems to imply that clicking it will cause the image to be deleted entirely, rather than just hidden from the current view.

Change History (4)

comment:1 follow-up: @nacin2 years ago

It is designed to only remove the header image, not the underlying upload.

But: uploads should be disabled in trunk. What dark voodoo is this?

comment:2 @pento2 years ago

The wording could probably be better. For me, at least, I assumed it was going to remove from the Media Library.

You just made me svn info and svn up to be sure. I'm definitely testing on trunk HEAD, with Twenty Eleven. I see a shiny upload box for me to drag and drop things on to.

comment:3 in reply to: ↑ 1 @SergeyBiryukov2 years ago

Replying to nacin:

But: uploads should be disabled in trunk.

For header images (#21355), but not for custom background.

comment:4 @nacin2 years ago

  • Version changed from trunk to 3.4

I'm happy to improve the string, but I don't see much room for improvement. We are usually pretty good about using "Delete" when there is a destructive action, and "Remove" when there is not. Also, we have used "Remove Image" on the Appearance > Header and Background screens probably since they were introduced.

Note: See TracTickets for help on using tickets.