WordPress.org

Make WordPress Core

Changes between Version 1 and Version 2 of Ticket #25357, comment 3


Ignore:
Timestamp:
09/21/2013 06:52:43 AM (7 years ago)
Author:
DrewAPicture
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #25357, comment 3

    v1 v2  
    22
    33Some notes on [attachment:25357.diff]:
    4 * `install_plugins_tabs` filter: "Filter the tabs that are shown on the page." needs to be more specific. It's best to document hooks as if they are independent of each other. So maybe, "Filter the tabs that are show on the Plugin Install screen." For the @param description, maybe "An array of tabs used on the Plugin Install screen." or something
     4* `install_plugins_tabs` filter: "Filter the tabs that are shown on the page." needs to be more specific. It's best to document hooks as if they are independent of each other. So maybe, "Filter the tabs that are shown on the Plugin Install screen." For the @param description, maybe "An array of tabs used on the Plugin Install screen." or something
    55* `install_plugins_nonmenu_tabs` filter: "Filter the $nonmenu_tabs." isn't very helpful information. Perhaps something like "Filter tabs not associated with a menu item." Same for the @param description. I know it's copied from the function, but it's kind of vague and unhelpful. Try to be clear and direct.
    66* `install_plugins_table_api_args_$tabs` filter: "Plugin API" should be "Plugin Installer API", "the $tab." should be "each tab."