Closed
Bug 260562
Opened 18 years ago
Closed 4 years ago
accessibility.typeaheadfind.enabletimeout is ignored
Categories
(Toolkit :: Find Toolbar, defect, P2)
Toolkit
Find Toolbar
Tracking
()
RESOLVED
FIXED
mozilla63
Tracking | Status | |
---|---|---|
firefox63 | --- | fixed |
People
(Reporter: spamcop, Assigned: spillner)
References
(Depends on 1 open bug)
Details
(Keywords: access)
Attachments
(1 file)
Find as you type doesn't work on Macs. The new, cute search bar on the bottem immediately goes away after I typed only one letter and the next letter immidiately starts a new search, no matter how fast I type. I set accessibility.typeaheadfind.enabletimeout to false, so it should stay there forever, but it goes away immediately
Oh, my current browser version, of course: Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.7.3) Gecko/20040919 Firefox/0.10
Updated•18 years ago
|
Component: Search → Find Toolbar / FastFind
Updated•18 years ago
|
Assignee: p_ch → firefox
QA Contact: firefox.search
Comment 2•18 years ago
|
||
Works for me in build 20040920 (branch) Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.7.3) Gecko/20040920 Firefox/0.10
Jo, can you do me a favor? Can you open a new tab and enter "about:config" as address. Then please enter "typea" into the filter line. Some lines will be left over. Can you please post all the settings left over here? I'd like to compare them to my settings. Maybe this explains why it works for you but not for me. Maybe it's a setting problem. But then it's still a bug IMHO. No setting should cause such an incorrect behavior.
Okay, I made some tests myself. The problem is, that accessibility.typeaheadfind.enabletimeout is ignored on the Mac (or it is ignored in general. Have to test it under Windows this evening). The setting accessibility.typeaheadfind.timeout was set to 0 in my config; this was the way how to disable timout in earlier versions. But now 0 really means a timeout of 0, so it times out at once. May I suggest a solution for this problem? A timeout of 0 should still be treated as "no-timeout", since it makes no sense to really have a timeout of 0. And accessibility.typeaheadfind.enabletimeout should be honored when set to false, regardless of timeout set.
Summary: Find as you type doesn't work any longer on Macintosh → accessibility.typeaheadfind.enabletimeout is ignored
Confirmed in Windows (XP + SP2) Assuming it applies to all platforms
OS: MacOS X → All
Comment 6•18 years ago
|
||
this bug is fixed with the patch in bug 265915
We can close it once the patch has been checked in and I can verify correct behavior in one of the nightly builds
Depends on: 265915
Comment 8•18 years ago
|
||
Confirmed. Running Windows XP Pro SP2, tested with Firefox 1.0.1 and latest Trunk with clean profiles.
Same thing here (Mozilla/5.0 (Windows; U; Windows NT 5.1; fr-FR; rv:1.7.6) Gecko/20050223 Firefox/1.0.1) Find as you type dialog box just appears a microsecond. accessibility.typeaheadfind.timeout=5000 accessibility.typeaheadfind.enabletimeout=true Note the Ctrl-F shortcut or menu action just works find : dialog box stay active indefinitely
Comment 10•17 years ago
|
||
This is still broken in Firefox 1.5 RC3. The find bar disappears after 5 seconds even when accessibility.typeaheadfind.enabletimeout is false. I'm running Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.8) Gecko/20051111 Firefox/1.5 on Windows XP SP2. I did a clean install of Firefox 1.5 RC3, not an upgrade, no old preferences or settings. Extensions I'm using: DOM Inspector, Talkback, Google Toolbar for Firefox Workaround: Set accessibility.typeaheadfind.timeout to 999999999 and restart Firefox.
Updated•17 years ago
|
Assignee: firefox → nobody
QA Contact: fast.find
Hardware: Macintosh → All
Version: unspecified → Trunk
Updated•14 years ago
|
Product: Firefox → Toolkit
Comment 11•8 years ago
|
||
20/9/2004: please fix this bug! Thank you!
Updated•6 years ago
|
Priority: -- → P2
Assignee | ||
Comment 12•4 years ago
|
||
Mike DeBoer correctly noted in a comment at https://bugzilla.mozilla.org/show_bug.cgi?id=254592 that enabletimeout is no longer used and should be removed. I updated the timeout logic to treat a zero or negative value as effectively "no automatic timeout" for the quick-find dialog (otherwise, setting the timeout value to a small or negative value makes the feature unusable). This is a corollary to the bugfix at https://phabricator.services.mozilla.com/D3404 ; I've split it out into a separate patch to avoid confusing that issue. Update: this specific issue already had its own bug at https://bugzilla.mozilla.org/show_bug.cgi?id=260562, and another mention at https://bugzilla.mozilla.org/show_bug.cgi?id=265915 .
Comment 13•4 years ago
|
||
Comment on attachment 9001904 [details]
Remove obsolete accessibility.typeaheadfind.enabletimeout preference.
Kris Maglione [:kmag] has approved the revision.
Attachment #9001904 -
Flags: review+
Comment 14•4 years ago
|
||
https://treeherder.mozilla.org/#/jobs?repo=try&revision=4dbfc1aba4822d0784ef9d31ac289b98fb3c1ebf
Updated•4 years ago
|
Assignee: nobody → spillner
Keywords: checkin-needed
Comment 15•4 years ago
|
||
With Diff 8711 on Sat, August 18, 2018, 1:25 AM GMT+3, by dvarga@mozilla.com. Error: We're sorry, Autoland could not rebase your commits for you automatically. Please manually rebase your commits and try again. (255, 'applying /tmp/tmpmwiPga\npatching file toolkit/content/widgets/findbar.xml\nHunk #1 succeeded at 335 with fuzz 2 (offset -3 lines).\nHunk #2 FAILED at 373\n1 out of 3 hunks FAILED -- saving rejects to file toolkit/content/widgets/findbar.xml.rej\nabort: patch failed to apply', '')
Flags: needinfo?(spillner)
Keywords: checkin-needed
Assignee | ||
Comment 16•4 years ago
|
||
(In reply to Cosmin Sabou [:CosminS] from comment #15) Rebased, should succeed now.
Flags: needinfo?(spillner)
Comment 17•4 years ago
|
||
(In reply to spillner from comment #16) > (In reply to Cosmin Sabou [:CosminS] from comment #15) > > Rebased, should succeed now. In the future, please add the checkin-needed keyword when something is ready to land :)
Keywords: checkin-needed
Comment 18•4 years ago
|
||
Pushed by apavel@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/f4f56776a112 Remove obsolete accessibility.typeaheadfind.enabletimeout preference. r=kmag
Keywords: checkin-needed
Comment 19•4 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/f4f56776a112
Status: NEW → RESOLVED
Closed: 4 years ago
status-firefox63:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla63
You need to log in
before you can comment on or make changes to this bug.
Description
•