Opened 21 years ago
Closed 18 years ago
#2152 closed defect (bug) (wontfix)
Two search forms on one page invalidates XHTML 1.0 standards
| Reported by: | abhay | Owned by: | pishmishy |
|---|---|---|---|
| Priority: | normal | Milestone: | |
| Component: | General | Version: | 2.0 |
| Severity: | minor | Keywords: | |
| Cc: | Focuses: |
Description
Two examples:
- Blog with no posts causes a second search form to appear in the content area with the default Kubrick theme in addition to the search box already in the sidebar. (Although this is really trivial, I would say that this is still worth fixing.)
- Search yeilding no results produces two search boxes as above. This is more relevant.
Fix would be to revert to the old style where we had the "Search yielded no results" message.
Attachments (2)
Change History (12)
#4
follow-up:
↓ 5
@
19 years ago
- Keywords has-patch removed
- Milestone 2.2 → 2.4
I don't like this fix. Many themes duplicate this style of providing a search box when no results are found, without having to find it in the sidebar.
#5
in reply to: ↑ 4
;
follow-up:
↓ 6
@
19 years ago
Replying to foolswisdom:
I don't like this fix. Many themes duplicate this style of providing a search box when no results are found, without having to find it in the sidebar.
What other options are there when you're only allowed to have 1 form on a page?
#6
in reply to: ↑ 5
@
19 years ago
Replying to Nazgul:
What other options are there when you're only allowed to have 1 form on a page?
Showing a search box when no results is a common experience, so the solution should account for that.
#8
@
19 years ago
- Owner changed from to
- Status new → assigned
Personally I don't think that the default theme should ever display more than one search box on any page. It's unnecessary interface clutter.
If the appearance of the side bar never changed between pages throughout the theme I could accept an argument for leaving it for reasons of consistency. But that doesn't appear to be the case.
Patching the theme so that the sidebar search box has a different name is trivial if that turns out to be the preferred option.
Neither change effects what authors of other themes want to do.
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
Changes would be to remove this line from wp-content/themes/default/search.php and wp-content/themes/default/index.php :