[wp-trac] [WordPress Trac] #13053: Remove searchform.php from Twenty Ten

WordPress Trac wp-trac at lists.automattic.com
Mon Apr 19 20:45:03 UTC 2010


#13053: Remove searchform.php from Twenty Ten
-------------------------+--------------------------------------------------
 Reporter:  demetris     |       Owner:           
     Type:  enhancement  |      Status:  new      
 Priority:  normal       |   Milestone:  3.0      
Component:  Themes       |     Version:  3.0      
 Severity:  normal       |    Keywords:  has-patch
-------------------------+--------------------------------------------------
 In the interests of simplicity and of taking advantage of—rather than
 duplicating—useful functionality provided by WP core, I propose to
 completely remove searchform.php from Twenty Ten.

 The differences of Twenty Ten’s searchform.php from the output of
 {{{get_search_form}}} are two:

  1. Twenty Ten omits the {{{role}}} attribute.  There is no reason to do
 that anymore, since ARIA landmark roles are valid in HTML5 and are
 recognized as such by the W3 validator.

  2. Twenty Ten adds the string “Search” before the search box.  That
 string is redundant.

 I don’t know how to make a patch for the propesed change.  Simply delete
 the file! :-)

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/13053>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list