[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