Closed Bug 61417 Opened 24 years ago Closed 24 years ago

Clicking "search" sidebar tab should set focus to textbox

Categories

(SeaMonkey :: Search, defect, P1)

x86
Windows 98

Tracking

(Not tracked)

VERIFIED FIXED
mozilla0.8

People

(Reporter: jruderman, Assigned: matt)

References

Details

Attachments

(1 file)

Clicking on the "search" tab in the sidebar should set focus to the textbox in 
the search panel so that the user doesn't have to click on the tab, and then 
click on the textbox.  My guess is that this would be fixed by setting up the 
xul equivalent of a body-onload event to do document.getElementById("sidebar-
search-text").focus.

This bug depends on:
bug 50223 Clicking on Sidebar tab should set focus to sidebar page or widget

This bug probably also depends these bugs, which are present at least for html 
dom:
bug 41813 [Linux, normal] and 
bug 50665 [Windows, enhancement]
textbox.focus() shouldn't grab focus from other mozilla windows


I need to do this for my "calculator" panel as well 
(http://www.cs.hmc.edu/~jruderma/js/calc/).
Depends on: 41813, 50223, 50665
[This bug doesn't depend on the textbox.focus bugs. They would make the fixing of 
this bug slightly more annoying, but they certainly aren't stopping it from being 
fixed.]
No longer depends on: 41813, 50665
Keywords: nsbeta1
Netscape Nav triage team: this is a Netscape beta stopper.
Priority: P3 → P1
Target Milestone: --- → mozilla0.8
Attached patch patchSplinter Review
fix checked in
r=law
sr=ben
Status: NEW → RESOLVED
Closed: 24 years ago
Resolution: --- → FIXED
erik@mint.com said on bug 9086 that new Navigator windows didn't focus 
anything, and it turned out that his sidebar was collapsed and the Search 
textbox was grabbing focus away from the first page opened in each new window.  
I hope bug 41813, bug 50665, and bug 50223 get fixed soon :)
The fix for bug 41813 (textbox.focus() shouldn't steal focus from other 
windows) didn't fix the sidebar case, so I filed bug 76621.
mass-verifying claudius' Fixed bugs which haven't changed since 2001.12.31.

if you think this particular bug is not fixed, please make sure of the following
before reopening:

a. retest with a *recent* trunk build.
b. query bugzilla to see if there's an existing, open bug (new, reopened,
assigned) that covers your issue.
c. if this does need to be reopened, make sure there are specific steps to
reproduce (unless already provided and up-to-date).

thanks!

[set your search string in mail to "AmbassadorKoshNaranek" to filter out these
messages.]
Status: RESOLVED → VERIFIED
Product: Core → SeaMonkey
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: