Make WordPress Core

Opened 16 years ago

Closed 10 years ago

#9931 closed enhancement (wontfix)

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

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

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 16 years ago.
9931.diff (1.3 KB) - added by wojtek.szkutnik 14 years ago.

Download all attachments as: .zip

Change History (7)

#1 @Denis-de-Bernardy
16 years ago

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

@stgoos
16 years ago

#2 @Denis-de-Bernardy
16 years ago

  • Keywords has-patch removed

#3 @wojtek.szkutnik
14 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
11 years ago

  • Component changed from Administration to Appearance
  • Focuses administration added

#5 @ocean90
10 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.