Closed Bug 76484 Opened 24 years ago Closed 23 years ago

Allow multiple QuickSearch forms in a single document

Categories

(Bugzilla :: Query/Bug List, defect, P1)

2.11
Other
Other
defect

Tracking

()

VERIFIED FIXED
Bugzilla 2.16

People

(Reporter: afranke, Assigned: jacob)

References

Details

The QuickSearch documentation should be able to present examples the same way Google does, so that the user can modify them and/or submit them directly from the documentation page. See http://www.google.com/help/basics.html and http://www.google.com/help/refinesearch.html for example pages. For this to work, it must be possible to have multiple QuickSearch forms in a single document. This is currently impossible because the functions in quicksearch.js don't take any arguments, they read the value from a hardcoded location instead (document.f.id.value or something like this). This has to be changed so that a) the value or b) the textbox form element or c) the form itself is passed as an argument to the JS function. - a) and b) are very similar; the only difference is that - An advantage of c) is that the interface would not need to be changed in case QuickSearch is extended by additional, optional form elements (radio buttons, checkboxes, ...), so that quicksearch.js could support these new elements but still be backwards compatible. - A disadvantage of c) is that name="id" for the textbox would still have to be hardcoded. Feel free to take this one if you like. Currently I have no time, but maybe in two weeks...
Target Milestone: --- → Future
I have seen Jake working on something that looks like option a). See bug 37339, attachment 40917 [details] [diff] [review] ("updated patch to quicksearch", 2001-07-28). Anyway, this is the next thing on my list. Reassigning to Jake in the hope that we can do this together. (Feel free to push it back to me or nobody if you don't want it.) By the way, quicksearchhack.html probably needs the same change as quicksearch.html and index.html.
Assignee: nobody → jake
Yes, quicksearchhack.html will need that change, too. http://landfill.tequilarista.org/bz37339/ is currently running w/this patch on it, so feel free to test it there.
Blocks: 37339
Priority: -- → P3
Target Milestone: Future → Bugzilla 2.16
Status: NEW → ASSIGNED
Component: Bugzilla → Query/Bug List
Product: Webtools → Bugzilla
Version: Bugzilla 2.11 → 2.11
blocks a P1, therefore it's a P1
Priority: P3 → P1
Based on comments, it looks like this was fixed when the patch on bug 37339 was checked in.
Status: ASSIGNED → RESOLVED
Closed: 23 years ago
Resolution: --- → FIXED
Verified fixed: index.tmpl now calls QuickSearch(f.id.value), see http://lxr.mozilla.org/mozilla/source/webtools/bugzilla/template/default/index.tmpl#67
Status: RESOLVED → VERIFIED
QA Contact: matty_is_a_geek → default-qa
You need to log in before you can comment on or make changes to this bug.