[wp-trac] [WordPress Trac] #18690: Change the display layout of help tabs

WordPress Trac wp-trac at lists.automattic.com
Tue Oct 4 15:45:49 UTC 2011


#18690: Change the display layout of help tabs
-------------------------------------------------+-------------------------
 Reporter:  jane                                 |       Owner:
     Type:  task (blessed)                       |  koopersmith
 Priority:  normal                               |      Status:  reviewing
Component:  UI                                   |   Milestone:  3.3
 Severity:  normal                               |     Version:  3.3
 Keywords:  has-patch dev-feedback needs-        |  Resolution:
  testing                                        |
-------------------------------------------------+-------------------------

Comment (by azaozz):

 Replying to [comment:37 dd32]:

 Yes, most screens will have to be converted to using WP_Screen, we will do
 that while revising the help text.

 Replying to [comment:38 GaryJ]:
 > You shouldn't precede an argument that has no default value, with one
 that has.

 Was 50/50 whether to swap the order of args or make the first one
 optional. Currently `$current_screen->add_option()` is used for both the
 built-in options (screen layout, table columns, etc.) and for the HTML for
 custom options. The first requires option_mane and an array, the second is
 just the HTML string.

 Alternatively we could split the method into two:
 `$current_screen->add_default_option($name, $settings)` and
 `$current_screen->add_custom_option($html)`.

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/18690#comment:40>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list