[wp-trac] [WordPress Trac] #20492: Improve submit_button() (was: Switch submit_button()'s parameters to an associative array)

WordPress Trac wp-trac at lists.automattic.com
Thu Apr 19 21:39:51 UTC 2012


#20492: Improve submit_button()
-----------------------------+------------------------------
 Reporter:  johnbillion      |       Owner:
     Type:  feature request  |      Status:  new
 Priority:  low              |   Milestone:  Awaiting Review
Component:  General          |     Version:  3.4
 Severity:  normal           |  Resolution:
 Keywords:                   |
-----------------------------+------------------------------
Changes (by johnbillion):

 * keywords:  has-patch needs-testing =>
 * type:  enhancement => feature request


Comment:

 Okay I've changed my mind on this. Clearly `submit_button()`'s default
 parameters are not appropriate 90% of the time it's used. Going through
 core to patch the calls to `submit_button()` I can see that the 'wrap'
 parameter is set to 'false' most of the time, and the 'type' parameter is
 set to 'button' or 'secondary' most of the time.

 We'd be better off introducing a couple of new submit button functions
 (one for each button type) and deprecating `submit_button()`.

 I don't really want to waste any more time on the issue though, so someone
 else can take this on if they like.

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


More information about the wp-trac mailing list