Note: There are a few cases of duplicates in user autocompletion which are being worked on.

Internet Keywords should be off in Mozilla by default

VERIFIED FIXED in mozilla0.9.5

Status

()

Core
Networking
VERIFIED FIXED
16 years ago
14 years ago

People

(Reporter: benc, Assigned: gerv)

Tracking

Trunk
mozilla0.9.5
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

(Reporter)

Description

16 years ago
This is NOT a dupe of bug 76547. That is a discussion about moving of "Internet
Kewords" from the Netscape commercial build.

I think this should be off for several reason:

1- The text says "Mozilla Internet Keywords" in the prefs, but there is no such
thing. If you hit the info button it goes to "http://www.mozilla.org".

2- This is a commercial feature, that is branded.

3- Mozilla has lots of DNS and name resolution problems that are currently 
hidden by this feature. For example, bug 95707 really shows how out of date our
resolution architecture is... mozilla can't even find... "mozilla".

This bug is also relate to the variety of bugs I have filed asking for that
would improve the robustness and (for lack of a better word...) modernity of our
ability to take stuff users type click on and convert it to the sensible target.

Bug 88217 especially, but also issues about handling incomplete URLs that don't
have an fqdn.

Comment 1

16 years ago
Mozilla builds are for testing and development purposes. This feature confuses
connectivity testing. It should be disabled by default in our testing builds. It
would also be useful, for those interested in testing this specific feature, if
the button actually pointed to a document explaining what this feature does and
how it can be configured. With these two changed I think we can make testing a
lot more clear. 
(Reporter)

Comment 2

16 years ago
When you turn off keywords, you see the change in prefs.js.

user_pref("keyword.enabled", false);

In all.js, you see:

pref("keyword.URL", "http://keyword.netscape.com/keyword/");
pref("keyword.enabled", true);

in Netscape 6.1, ns-all.js already has:

pref("keyword.enabled", true);

I can't find a pref of the "More Info" button's URL, so that is probably in the
panel.

I'm hacking my prefs now to log how the queries are sent.
Summary: Prefs > Navigator > Smart Browsing > Internet Keyword - should be off in Mozilla by default → Prefs > Navigator > Smart Browsing > Internet Keywords - should be off in Mozilla by default
(Reporter)

Comment 3

16 years ago
I modified my server to accept requests and forward them to the normal URL.

The keyword queries are simply the URL encoded string:

"quote aol" -> "quote%20aol" after the keyword URL.
(Assignee)

Comment 4

16 years ago
OK, can someone please tell me what the URL for "more information" is in
Netscape builds (to give me something to work from for a Mozillified version)?

I'll then change the URL for that button, and change the pref in all.js to
false. I assume that's all this bug is about, right? :-) Some NS person should
probably turn it back on in ns-prefs.js.

Gerv
(Reporter)

Comment 5

16 years ago
I'd finish the mozilla version of the document tonight, and include the netscape
URL for the "more info" button in my write-up.
(Assignee)

Comment 6

16 years ago
OK. I have a patch to turn this off by default, and point to the explanatory
document, which I've checked in as:
http://www.mozilla.org/projects/seamonkey/internet-keywords.html

The keyword.enabled and keyword.URL prefs are already in all-ns.js, and so it
seems we don't actually need any coordination with Netscape.

Review requested :-)

Gerv
Assignee: neeti → gerv
OS: Windows 98 → All
Hardware: PC → All
Summary: Prefs > Navigator > Smart Browsing > Internet Keywords - should be off in Mozilla by default → Internet Keywords should be off in Mozilla by default
Target Milestone: --- → mozilla0.9.5
(Assignee)

Comment 7

16 years ago
Created attachment 51563 [details] [diff] [review]
Patch v.1
(Assignee)

Comment 8

16 years ago
Created attachment 51569 [details] [diff] [review]
Patch v.2 - changed URL

Comment 9

16 years ago
Comment on attachment 51569 [details] [diff] [review]
Patch v.2 - changed URL

r=doron
Attachment #51569 - Flags: review+
(Assignee)

Updated

16 years ago
Blocks: 101793
Comment on attachment 51569 [details] [diff] [review]
Patch v.2 - changed URL

sr=shaver
Comment on attachment 51569 [details] [diff] [review]
Patch v.2 - changed URL

sr=shaver
Attachment #51569 - Flags: superreview+
(Assignee)

Comment 12

16 years ago
Checked in.

Gerv
Status: NEW → RESOLVED
Last Resolved: 16 years ago
Resolution: --- → FIXED
(Reporter)

Comment 13

15 years ago
VERIFIED: 
I've been testing this default in mozilla 1.0 and later.
Status: RESOLVED → VERIFIED
(Reporter)

Comment 14

14 years ago
There are a few scattered reports of people who want a default behavior in
mozilla, where this is on by default. I've duped those bugs into Bug 197918 to
track the feedback.
You need to log in before you can comment on or make changes to this bug.