WordPress.org

Make WordPress Core

Opened 11 years ago

Closed 5 years ago

#9931 closed enhancement (wontfix)

Extra class for the wrap div's in custom-header.php

Reported by: stgoos Owned by:
Milestone: Priority: normal
Severity: normal Version:
Component: Customize Keywords: has-patch needs-testing gsoc
Focuses: administration Cc:
PR Number:

Description

I'm using custom-header.php to complete the options section for my own themes but for some themes I only need the image section to be visible>

Would it be possible to add an extra class reference to the <div class="wrap"> sections? This would make it possible for me (and others) to not display the irrelevant sections (and thus avoid questions from users about why changes aren't reflected).

My suggestions is to go for something like this:

<div class="wrap custom-header-text">
<div class="wrap custom-header-image">
<div class="wrap custom-header-reset">

And perhaps that:

<div class="wrap custom-header-text">
<?php screen_icon(); ?>

needs to be changed into something like:

<?php screen_icon(); ?>
<div class="wrap custom-header-text">

to keep the icon available when you choose to hide the text section.

Cheers :)

Attachments (2)

custom-header.php (13.2 KB) - added by stgoos 11 years ago.
9931.diff (1.3 KB) - added by wojtek.szkutnik 9 years ago.

Download all attachments as: .zip

Change History (7)

#1 @Denis-de-Bernardy
11 years ago

  • Component changed from General to Administration
  • Keywords needs-patch added
  • Milestone changed from Unassigned to Future Release

@stgoos
11 years ago

#2 @Denis-de-Bernardy
11 years ago

  • Keywords has-patch removed

@wojtek.szkutnik
9 years ago

#3 @wojtek.szkutnik
9 years ago

  • Cc wojtek.szkutnik@… added
  • Keywords has-patch needs-testing gsoc added; needs-patch custom header class removed

Adapted the suggestions to current code.

#4 @jeremyfelt
6 years ago

  • Component changed from Administration to Appearance
  • Focuses administration added

#5 @ocean90
5 years ago

  • Milestone Future Release deleted
  • Resolution set to wontfix
  • Status changed from new to closed

See #25571.

Note: See TracTickets for help on using tickets.