Make WordPress Core

Opened 14 years ago

Closed 12 years ago

#17032 closed enhancement (wontfix)

Twenty Ten Theme - Widget Enhancement

Reported by: rozani's profile rozani Owned by:
Milestone: Priority: normal
Severity: normal Version: 3.1
Component: Bundled Theme Keywords:
Focuses: Cc:

Description

separate twentyten_widget_innit into twentyten_sidebar_widget_innit and twentyten_footer_widget_innit for the ease of child theme creation

Attachments (1)

widget.diff (6.6 KB) - added by rozani 14 years ago.

Download all attachments as: .zip

Change History (5)

#1 @rozani
14 years ago

separate twentyten_widget_innit into twentyten_sidebar_widget_innit and twentyten_footer_widget_innit for the ease of child theme creation.

Draftly, modification as below. For details, see attachment.

Original widget function:

function twentyten_widgets_init() {

Separate widget function to sidebar and footer

function twentyten_sidebar_widgets_init() {
function twentyten_footer_widgets_init() {

Also, change the innit function.
Originally.

add_action( 'widgets_init', 'twentyten_widgets_init' );
{{{

Separate innit to sidebar innit and footer innit.
}}}
add_action( 'widgets_init', 'twentyten_sidebar_widgets_init' );
add_action( 'widgets_init', 'twentyten_footer_widgets_init' );

@rozani
14 years ago

#2 @kawauso
14 years ago

Wouldn't this break any existing child themes expecting 'twentyten_widgets_init' ?

#3 @SergeyBiryukov
12 years ago

  • Component changed from General to Bundled Theme

#4 @lancewillett
12 years ago

  • Milestone Awaiting Review deleted
  • Resolution set to wontfix
  • Status changed from new to closed

It's better to have them on the same action. I think we shouldn't make a change like this.

Note: See TracTickets for help on using tickets.