Make WordPress Core

Opened 12 years ago

Closed 11 years ago

Last modified 11 years ago

#22333 closed enhancement (fixed)

Add IDs to .misc-pub-section divs in meta-boxes.php

Reported by: nofearinc's profile nofearinc Owned by: helen's profile helen
Milestone: 3.7 Priority: normal
Severity: normal Version: 3.4
Component: Administration Keywords: 3.7-early has-patch commit
Focuses: Cc:

Description

In wp-admin/includes/meta-boxes.php the post_submit_meta_box has several sections with class .misc-pub-section. Should you have to style/alter any of them, it's not consistent as:

  1. first div has only class="misc-pub-section"
  2. second div has class="misc-pub-section" id="visibility"
  3. third div has class="misc-pub-section curtime"

Since this is a unique box on the screen, I've added two more IDs to div blocks 1 and 3 to allow for easier styling changes.

Attachments (3)

meta-boxes.php.patch (1.0 KB) - added by nofearinc 12 years ago.
meta-boxes.php patch for IDs
22333.2.patch (2.3 KB) - added by nofearinc 11 years ago.
adding classes to all misc boxes
22333.3.patch (6.4 KB) - added by nofearinc 11 years ago.
Adding few extra classes in other files, removing .num-revisions

Download all attachments as: .zip

Change History (12)

@nofearinc
12 years ago

meta-boxes.php patch for IDs

#1 @SergeyBiryukov
11 years ago

  • Version changed from trunk to 3.4

#2 @markoheijnen
11 years ago

  • Keywords 3.7-early commit added

#3 @nacin
11 years ago

Would be nice to just give these descriptive classes. IDs aren't really a necessity.

Also, there's a new .misc-pub-section in core.

#4 @SergeyBiryukov
11 years ago

  • Keywords has-patch needs-refresh added; commit removed

@nofearinc
11 years ago

adding classes to all misc boxes

#5 @nofearinc
11 years ago

Updated the patch with classes for all pub sections. I've used the misc-pub prefix for consistency across all boxes similarly to the misc-pub-section class name itself.

#6 @wonderboymusic
11 years ago

  • Milestone changed from Awaiting Review to 3.7

these are all marked 3.7-early

@nofearinc
11 years ago

Adding few extra classes in other files, removing .num-revisions

#7 @nacin
11 years ago

  • Keywords commit added; needs-refresh removed

#8 @helen
11 years ago

  • Owner set to helen
  • Resolution set to fixed
  • Status changed from new to closed

In 25083:

Add more descriptive classes to submit meta box sections. props nofearinc. fixes #22333.

#9 @helen
11 years ago

In 25297:

Add dynamic classes for attachment metadata submitbox sections. see #22333, #25171.

Note: See TracTickets for help on using tickets.