Closed
Bug 207601
Opened 22 years ago
Closed 22 years ago
UI to disable Find As You Type (type-ahead find)
Categories
(Firefox :: Settings UI, enhancement)
Firefox
Settings UI
Tracking
()
VERIFIED
FIXED
People
(Reporter: mozilla7, Assigned: mconnor)
Details
Attachments
(1 file, 6 obsolete files)
3.00 KB,
patch
|
noririty
:
review+
|
Details | Diff | Splinter Review |
User-Agent: Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.4b) Gecko/20030516 Mozilla Firebird/0.6
Build Identifier: Mozilla/5.0 (Macintosh; U; PPC Mac OS X Mach-O; en-US; rv:1.4b) Gecko/20030516 Mozilla Firebird/0.6
I personally find type-ahead find to be annoying, since accidentally pressing a
key tends to scroll unpredictably and I have to figure out where I was. It can
be disabled, but not from the Preferences dialog.
Reproducible: Always
Steps to Reproduce:
1. Open Preferences
2. Look for a way to disable type-ahead find
Actual Results:
There is no such option.
Expected Results:
Perhaps it should be listed under General? Or perhaps a new category.
-> RFE
Mozilla Firebird Help has a solution.
http://texturizer.net/firebird/tips.html#beh_taf
Don't let Mozilla Firebird bloated by mess of options.
So I suggest WONTFIX.
Severity: normal → enhancement
Reporter | ||
Comment 2•22 years ago
|
||
Whoops, sorry, should have been an RFE. Fixing subject.
My concern is that the feature may be confusing to a new user, and it's not at all obvious that the
feature is called Find As You Type (I didn't realize it had been renamed myself - saw the
documentation but I guess I skipped over the introduction). Without knowing that, it's non-trivial
to find it in the documentation. I agree that Preferences shouldn't become unnecessarily bloated,
but I think this is reasonable.
Personally I would prefer that it be turned off by default, and let users turn it on - this gives them
a chance to see what it's called, so they can read documentation about how to use it if they want.
This is how Safari's tabbed browsing feature works, for example - it could confuse new users, so
it's off by default with an obvious option in Preferences. Getting it turned off by default is
probably a losing battle though, so I won't fight it. :-)
One definition of a bug is "a feature that can't be turned off". A feature that can't be turned off
except by hacking JavaScript code is better, but not as good as it could be. :-)
Summary: Need UI to disable type-ahead find → [RFE] UI to disable Find As You Type (type-ahead find)
![]() |
||
Comment 3•22 years ago
|
||
don't use RFE keyword. use enh severity
Summary: [RFE] UI to disable Find As You Type (type-ahead find) → UI to disable Find As You Type (type-ahead find)
![]() |
Assignee | |
Comment 4•22 years ago
|
||
taking QA contact, sorry about the bugspam
QA Contact: asa → mconnor
Comment 5•22 years ago
|
||
Comment 6•22 years ago
|
||
this js file should be placed next to pref-advanced.xul
Comment 7•22 years ago
|
||
Attachment #130022 -
Attachment is obsolete: true
![]() |
Assignee | |
Comment 8•22 years ago
|
||
Just because six prefs exist doesn't mean they all need to have UI
Links only and on/off are all that end-users are likely to use. The rest is
stuff more suited for hidden prefs.
Comment 9•22 years ago
|
||
this patch only includes UI options for on/off, text/links only, and sound
on/off. I thought it would be good idea to keep the sound pref as i have heard
some people complaining about it.
Attachment #130021 -
Attachment is obsolete: true
Comment 10•22 years ago
|
||
Attachment #130026 -
Attachment is obsolete: true
![]() |
Assignee | |
Comment 11•22 years ago
|
||
simpler patch, for eight lines of JS we can hold off on adding another file for
now, drop the sounds pref since its not going to bother 95% of people
Attachment #130087 -
Attachment is obsolete: true
Attachment #130088 -
Attachment is obsolete: true
![]() |
Assignee | |
Comment 12•22 years ago
|
||
move options to accessibility, clean up wording
![]() |
Assignee | |
Updated•22 years ago
|
Attachment #130128 -
Attachment is obsolete: true
Comment 14•22 years ago
|
||
Comment on attachment 130131 [details] [diff] [review]
revised patch
r=noririty
Attachment #130131 -
Flags: review+
Comment 15•22 years ago
|
||
fix landed
Status: NEW → RESOLVED
Closed: 22 years ago
Resolution: --- → FIXED
![]() |
Assignee | |
Comment 17•19 years ago
|
||
sorry for bugspam, long-overdue mass reassign of ancient QA contact bugs,
filter on "beltznerLovesGoats" to get rid of this mass change
QA Contact: mconnor → preferences
You need to log in
before you can comment on or make changes to this bug.
Description
•