WordPress.org

Make WordPress Core

Opened 4 years ago

Closed 4 years ago

Last modified 4 years ago

#25808 closed defect (bug) (fixed)

Twenty Fourteen: option 'display' as in wp_title for description of site

Reported by: michelwppi Owned by: SergeyBiryukov
Milestone: 3.8 Priority: normal
Severity: normal Version: 3.8
Component: Bundled Theme Keywords:
Focuses: Cc:

Description

The first test to enhance 2014 as multilingual child theme (as I done for 2010 to 2013) is (very) good BUT
in sidebar.php to display $description you forgot to add second parameter 'display' in

get_bloginfo ('description');

as done in wp_title...

get_bloginfo ('description', 'display');

By using this parameter, get_bloginfo is filterable (apply_filters at end of function) as wp_title used in header...

I think it will be coherent to be done in parent 2014.

M - xili-language author

Attachments (1)

25808.patch (544 bytes) - added by SergeyBiryukov 4 years ago.

Download all attachments as: .zip

Change History (4)

#1 @SergeyBiryukov
4 years ago

  • Component changed from Themes to Bundled Theme
  • Version set to trunk

#2 @SergeyBiryukov
4 years ago

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

In 26444:

Twenty Fourteen: use display context for site description in sidebar.php.

props michelwppi.
fixes #25808.

#3 @SergeyBiryukov
4 years ago

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