Opened 16 years ago
Closed 16 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
roleattribute. 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