[wp-trac] [WordPress Trac] #13207: Twenty Ten: searchform.php

WordPress Trac wp-trac at lists.automattic.com
Sun May 2 00:14:21 UTC 2010


#13207: Twenty Ten: searchform.php
-------------------------+--------------------------------------------------
 Reporter:  zeo          |       Owner:             
     Type:  enhancement  |      Status:  new        
 Priority:  normal       |   Milestone:  3.0        
Component:  Themes       |     Version:  3.0        
 Severity:  normal       |    Keywords:  2nd-opinion
-------------------------+--------------------------------------------------
 `get_search_form()` default:

 `<form role="search" method="get" id="searchform"
 action="http://localhost/" >
   <div>
     <label class="screen-reader-text" for="s">Search for:</label>
     <input type="text" value="" name="s" id="s" />
     <input type="submit" id="searchsubmit" value="Search" />
   </div>
 </form>`

 Twenty Ten searchform.php:

 `<form id="searchform" name="searchform" method="get"
 action="http://localhost/">
   <div>
     <label for="s">Search</label>
     <input type="text" id="s" name="s" />
     <input type="submit" id="searchsubmit" value="Search" />
   </div>
 </form>`

 Rather than came up with useless patches for this, do we actually need
 2010 own search form template? see: [http://core.svn.wordpress.org/trunk
 /wp-content/themes/twentyten/searchform.php searchform.php]

 The only different that I can see is the text for the label, role and
 additional class screen-reader-text (which I believe is missing from 2010
 searchform.php).

 Thoughts?

 Also please look: http://core.trac.wordpress.org/ticket/12961

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


More information about the wp-trac mailing list