Opened 15 years ago
Closed 15 years ago
#13053 closed enhancement (fixed)
Remove searchform.php from Twenty Ten
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | 3.0 | Priority: | normal |
Severity: | normal | Version: | 3.0 |
Component: | Themes | Keywords: | has-patch |
Focuses: | Cc: |
Description
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:
- 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.
- 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! :-)
Change History (3)
Note: See
TracTickets for help on using
tickets.
+1 delete