[wp-trac] [WordPress Trac] #28578: Re-add install_themes_tabs filter

WordPress Trac noreply at wordpress.org
Thu Jun 19 00:50:29 UTC 2014


#28578: Re-add install_themes_tabs filter
------------------------------------+-----------------------------
 Reporter:  ghost1227               |       Owner:
     Type:  defect (bug)            |      Status:  new
 Priority:  normal                  |   Milestone:  4.0
Component:  Themes                  |     Version:  trunk
 Severity:  normal                  |  Resolution:
 Keywords:  has-patch dev-feedback  |     Focuses:  administration
------------------------------------+-----------------------------
Changes (by SergeyBiryukov):

 * component:  General => Themes
 * milestone:  Awaiting Review => 4.0


Comment:

 1. The patch should keep the context on 'Browse' string: `_x( 'Browse',
 'themes' )` vs. `__( 'Browse' )`.
 2. [http://make.wordpress.org/core/handbook/inline-documentation-standards
 /php-documentation-standards/ Filter docs] are needed. Since
 `WP_Theme_Install_List_Table` is no longer used, I'd suggest copying the
 [source:tags/3.9.1/src/wp-admin/includes/class-wp-theme-install-list-
 table.php#L51 docblock] from there (with updated defaults) and replacing
 it with a [http://make.wordpress.org/core/handbook/inline-documentation-
 standards/php-documentation-standards/#4-1-duplicate-hooks duplicate hook
 comment] there.

--
Ticket URL: <https://core.trac.wordpress.org/ticket/28578#comment:4>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list