WordPress.org

Make WordPress Core

Opened 2 years ago

Last modified 6 weeks 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 10 months ago.
Add Current Header/Background image identifier
22857.2.diff (1.7 KB) - added by coreymcollins 3 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 3 months ago.
Removed redundant "Header Image" text when displaying "Current Header Image"

Download all attachments as: .zip

Change History (12)

comment:1 @johnbillion2 years ago

  • Description modified (diff)

Affects background images too.

comment:2 @SergeyBiryukov2 years ago

  • Component changed from Media to Appearance

comment:3 @deltafactory16 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 @deltafactory16 months ago

  • Cc jeff@… added

comment:5 @ocean9010 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.

@deltafactory10 months ago

Add Current Header/Background image identifier

comment:6 @deltafactory10 months ago

  • Keywords has-patch added; needs-patch removed

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

comment:7 @deltafactory10 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 @samuelsidler9 months ago

  • Keywords needs-testing added

@coreymcollins3 months ago

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

@coreymcollins3 months ago

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

comment:9 @celloexpressions6 weeks ago

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