[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