Opened 2 years ago
Closed 3 months ago
#17032 closed enhancement (wontfix)
Twenty Ten Theme - Widget Enhancement
| Reported by: |
|
Owned by: | |
|---|---|---|---|
| Priority: | normal | Milestone: | |
| Component: | Bundled Theme | Version: | 3.1 |
| Severity: | normal | Keywords: | |
| 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)
Change History (5)
Wouldn't this break any existing child themes expecting 'twentyten_widgets_init' ?
comment:3
SergeyBiryukov — 5 months ago
- Component changed from General to Bundled Theme
comment:4
lancewillett — 3 months 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.

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' );