Closed
Bug 424509
Opened 17 years ago
Closed 16 years ago
Location bar autocomplete favors "http://" over domain name starting with "h"
Categories
(Firefox :: Address Bar, defect, P2)
Firefox
Address Bar
Tracking
()
VERIFIED
FIXED
Firefox 3.1a2
People
(Reporter: moz, Assigned: Mardak)
References
Details
(Whiteboard: [Fx2-parity])
Attachments
(2 files, 2 obsolete files)
(deleted),
image/png
|
Details | |
(deleted),
patch
|
dietrich
:
review+
|
Details | Diff | Splinter Review |
Linux, 2008032104:
1) Open a new profile with a few bookmarks.
2) Go to http://heise.de
3) Type "h" in the location bar.
Result: Lots of results starting with "http://"
Expected result: Domains starting with "h" (e.g. heise.de) at the top of the list.
=> Give protocol part of the URL a very low priority in autocompletion.
Updated•17 years ago
|
Flags: blocking-firefox3?
Comment 2•17 years ago
|
||
The learning algorithm should fix this, mostly, but ignoring the protocol when matching against the URL is probably a good idea when the user has typed only *part* of http:// or https:// -- when they've typed it all, we should definitely match -- as is stuff in the eTLD list.
Flags: wanted-firefox3+
Flags: blocking-firefox3?
Flags: blocking-firefox3-
OS: Linux → All
Priority: -- → P2
Answer me this: Why would a user in their right mind ever try to autocomplete on http://? It will return every web site that user has ever visited that uses HTTP. It should match on the domain name, like if I want to look up a site in my history like http://www.http-how-to.com. It might have limited use in finding NON-http protocols.
I vote for a heuristic to drop the score of hits against the protocol http://. Likewise for TLDs like com, net, etc. More detail in Bug 424670.
Assignee | ||
Comment 4•17 years ago
|
||
FYI: If you go to the location bar right now and press down, it'll do a search with the current url... which likely has http://
Comment 5•17 years ago
|
||
Matching against http://, or not, have their pros and cons. If I type just http://, I would expect the top HTTP URLs in my whole history to drop down, in order of decreasing frecency, but not those with file:, https:, about: or chrome: at the start -- and I have quite a number of these four too.
And, Skewer, if you say I'm not of right mind -- I can live with that. :-)
Assignee | ||
Comment 6•17 years ago
|
||
Assignee | ||
Comment 7•17 years ago
|
||
This will be fixed by the patch in bug 424717.
Assignee: nobody → edilee
Depends on: 424717
Hardware: PC → All
Whiteboard: [Fx2-parity][has fix in bug 424717]
Assignee | ||
Comment 8•17 years ago
|
||
Assignee | ||
Comment 9•17 years ago
|
||
Does this build fix the issue for you?
https://build.mozilla.org/tryserver-builds/2008-03-25_22:08-edward.lee@engineering.uiuc.edu-title.tags.prefix.half/
Bug 424673 - Optimize AwesomeBar for correcting typos or removing terms that caused no results
Bug 422277 - assertions in nsNavHistoryAutocomplete ("not a UTF8 string", etc.)
Bug 422698 - different levels of URL decoding for address bar and autocomplete suggestions
Bug 424717 - Location bar autocomplete should be willing to complete to a URL with a different protocol
Bug 424509 - Location bar autocomplete favors "http://" over domain name starting with "h"
Bug 418257 - Show what part of which tags match for urlbar autocomplete
Bug 424216 - displaying filename/path instead of title for (unvisited?) bookmarks
Bug 392143 - show keywords as url bar autocomplete choices
Bug 249468 - Add all bookmark keywords to location bar autocomplete drop-down list
Bug 395161 - Make it possible to restrict the url bar autocomplete results to bookmarks/history entries and match only url/title/tags
Bug 420437 - Search and emphasize quoted strings with spaces
Bug 423942 - "Clear List" in download manager should be "Clear Current List" during the search
Bug 424557 - Allow AwesomeBar to default search only urls (or history/titles/bookmarks/tags)
Bug 423718 - Use native platform colors for URLs in the location bar autocomplete, make the line between rows lighter
Bug 424213 - URLs without corresponding title are displayed with a blank title (which isn't full-height)
Bug 415190 - Autocomplete results can be incorrectly sized (clipped)
Bug 414326 - Use DownloadUtils for software update downloads
Reporter | ||
Comment 10•17 years ago
|
||
Yes, this fixes the issue from comment #c0. Thank you!
Comment 11•17 years ago
|
||
How to search for URLs which -start- with the search term (maybe after "http://" or "http://www.") as in previous versions?
Assignee | ||
Comment 12•17 years ago
|
||
So you can't match on "http://", but you can match on "www." with this build:
https://build.mozilla.org/tryserver-builds/2008-03-26_08:12-edward.lee@engineering.uiuc.edu-title.tags.prefix.half/
Comment 13•17 years ago
|
||
it still shows all URLs which contain the string, not only these which start with the string.
Assignee | ||
Updated•17 years ago
|
Assignee | ||
Updated•17 years ago
|
Depends on: 422277
Whiteboard: [Fx2-parity][has fix in bug 424717] → [Fx2-parity][has patch][need dietrich review][need bug 422277][has more testcases in bug 422698, bug 424717]
Assignee | ||
Comment 14•17 years ago
|
||
This also fixes bug 422698 and bug 424717 both have their own test cases as well.
Attachment #311618 -
Attachment is obsolete: true
Attachment #313760 -
Flags: review?(dietrich)
Assignee | ||
Comment 15•17 years ago
|
||
All dependencies are clear, is wanted-firefox3, has 3 separate testcases ready to land with this.
Whiteboard: [Fx2-parity][has patch][need dietrich review][need bug 422277][has more testcases in bug 422698, bug 424717] → [Fx2-parity][has patch][need dietrich review][has more testcases in bug 422698, bug 424717]
Comment 16•16 years ago
|
||
I came across the same problem even for the letter 'p', which effectively means 'p' is now my shortcut for a list of my most recently viewed sites, regardless of the domain!
Assignee | ||
Comment 18•16 years ago
|
||
Cleanup the testcase to use the new compact format.
Attachment #313760 -
Attachment is obsolete: true
Attachment #331821 -
Flags: review?(dietrich)
Attachment #313760 -
Flags: review?(dietrich)
Comment 19•16 years ago
|
||
Comment on attachment 331821 [details] [diff] [review]
v1.1
r=me, thanks!
Attachment #331821 -
Flags: review?(dietrich) → review+
Assignee | ||
Comment 20•16 years ago
|
||
Status: NEW → RESOLVED
Closed: 16 years ago
Flags: in-testsuite+
Resolution: --- → FIXED
Whiteboard: [Fx2-parity][has patch][need dietrich review][has more testcases in bug 422698, bug 424717] → [Fx2-parity]
Target Milestone: --- → Firefox 3.1a2
Verified FIXED using:
Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.9.1a2pre) Gecko/2008080702 Minefield/3.1a2pre
Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.4; en-US; rv:1.9.1a2pre) Gecko/20080808020649 Minefield/3.1a2pre
Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US; rv:1.9.1a2pre) Gecko/20080808031528 Minefield/3.1a2pre
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•