WordPress.org

Make WordPress Core

Opened 3 years ago

Last modified 8 months ago

#22857 new enhancement

'Header Image' state isn't removed from images previously used as header image

Reported by: johnbillion Owned by:
Milestone: Future Release Priority: normal
Severity: normal Version: 3.4.2
Component: Customize Keywords: has-patch needs-testing
Focuses: ui, administration Cc:

Description (last modified by johnbillion)

Steps to reproduce:

  1. Set your site's header image from Appearance -> Header by uploading an image or choosing one from your media library.
  2. Repeat step 1.
  3. Visit the Media Library and note that your previous header image still says 'Header Image' next to it.

Not a 3.5 regression as it happens in 3.4.2 too.

Attachments (3)

22857.patch (1.3 KB) - added by deltafactory 16 months ago.
Add Current Header/Background image identifier
22857.2.diff (1.7 KB) - added by coreymcollins 9 months ago.
Above fixes and: removed redundant "Header Image" text on "Current Header Image"
Screenshot 2014-12-08 10.27.08.png (45.7 KB) - added by coreymcollins 9 months ago.
Removed redundant "Header Image" text when displaying "Current Header Image"

Download all attachments as: .zip

Change History (12)

comment:1 @johnbillion3 years ago

  • Description modified (diff)

Affects background images too.

comment:2 @SergeyBiryukov3 years ago

  • Component changed from Media to Appearance

comment:3 @deltafactory23 months ago

As used by get_uploaded_header_images() and on the Appearance -> Header page, the intention of the Header Image label indicates that the current theme had used it as a header image at one time. It's used to provide a list of previous images to select quickly.

What about a separate indicator via _media_states() that shows the "Current Header Image" status?

comment:4 @deltafactory23 months ago

  • Cc jeff@… added

comment:5 @ocean9016 months ago

  • Keywords needs-patch added
  • Type changed from defect (bug) to enhancement

deltafactory is right.

I like the idea of "Current Header Image", let's try this.

@deltafactory16 months ago

Add Current Header/Background image identifier

comment:6 @deltafactory16 months ago

  • Keywords has-patch added; needs-patch removed

Looks like I had worked on this back then.. Patch provided.

comment:7 @deltafactory16 months ago

  • Focuses ui administration added

As provided, the patch leaves the "Header Image" and adds "Current Header Image" which is redundant. Is there value to adding to the status, or replacing it?

comment:8 @samuelsidler15 months ago

  • Keywords needs-testing added

@coreymcollins9 months ago

Above fixes and: removed redundant "Header Image" text on "Current Header Image"

@coreymcollins9 months ago

Removed redundant "Header Image" text when displaying "Current Header Image"

comment:9 @celloexpressions8 months ago

  • Milestone changed from Awaiting Review to Future Release
Note: See TracTickets for help on using tickets.