Make WordPress Core

Opened 11 years ago

Closed 11 years ago

Last modified 11 years ago

#25889 closed enhancement (fixed)

Use selected() in touch_time(), page_template_dropdown(), parent_dropdown(), and add head docs

Reported by: meloniq's profile meloniq Owned by: wonderboymusic's profile wonderboymusic
Milestone: 3.9 Priority: normal
Severity: normal Version: 3.8
Component: General Keywords: has-patch
Focuses: docs, administration, template Cc:

Description

Use selected() in touch_time(), page_template_dropdown(), parent_dropdown(), and add head docs.

Suggested by @SergeyBiryukov in #25879 to separate this changes from adding filter hook.

The touch_time() function probably could be described better.

Attachments (2)

25889.patch (4.2 KB) - added by meloniq 11 years ago.
use selected(), add head docs
25889.2.patch (4.4 KB) - added by DrewAPicture 11 years ago.
Standardized docs

Download all attachments as: .zip

Change History (9)

@meloniq
11 years ago

use selected(), add head docs

#1 @SergeyBiryukov
11 years ago

  • Milestone changed from Awaiting Review to 3.8

#2 @matt
11 years ago

  • Milestone changed from 3.8 to Future Release

Hasn't been much activity here, let's look at it for after the 3.8 release.

#3 @nacin
11 years ago

  • Component changed from Template to General
  • Focuses docs administration template added
  • Milestone changed from Future Release to 3.9

@DrewAPicture
11 years ago

Standardized docs

#4 @DrewAPicture
11 years ago

25889.2.patch brings the inline docs in-line with the standards

#5 @wonderboymusic
11 years ago

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

In 27188:

Use selected() where appropriate in touch_time(), page_template_dropdown(), and parent_dropdown(). Also, add proper docs.

Props meloniq, DrewAPicture.
Fixes #25889.

#6 @TobiasBg
11 years ago

@wonderboymusic: Seems like you (accidentally?) added get_the_permalink() for #24164 in [27188]?
Well, as the discussion in #24164 agrees, closing that as fixed with [27188] should also be ok.

#7 @wonderboymusic
11 years ago

EEK. Removed in [27189].

Note: See TracTickets for help on using tickets.