Closed Bug 429531 Opened 16 years ago Closed 16 years ago

Location bar should show non-word-boundary matches below word-boundary matches


(Firefox :: Address Bar, defect, P2)




Firefox 3


(Reporter: Mardak, Assigned: Mardak)




(2 files, 1 obsolete file)

We prune results that don't match on word boundaries, so if we end up not matching anything, we show nothing. But it could be useful to show at least something if we don't have enough word-boundary results.
blocking-firefox3? as this is what beltzner wanted to grant blocking for bug 429498.
Flags: blocking-firefox3?
Whiteboard: [has patch][need dietrich review]
Attached patch v1 (obsolete) — Splinter Review
Simple patch, simple idea.

New pref to control whether or not we match on word boundaries always, or first try word boundary then try matching anywhere.

If we search through everything and want more results, continue searching but relax the condition of matching on word boundaries.

Works with the "use previous results" optimization as well.
Assignee: nobody → edilee
Attachment #316333 - Flags: review?(dietrich)
(The pref value of 2 to always match on word boundaries is mostly just for testing to make sure the word boundary logic still works. ;))
Attached image screenshot of v1
Screenshot of matching on word boundaries first and then trying to match anywhere.
Attachment #316349 - Flags: ui-review?(beltzner)
Whiteboard: [has patch][need dietrich review] → [has patch][need dietrich review][need bug 429498]
Attached patch v1.1Splinter Review
Same as v1 but updated some comments.
Attachment #316333 - Attachment is obsolete: true
Attachment #316351 - Flags: review?(dietrich)
Attachment #316333 - Flags: review?(dietrich)
This at most doubles the search time, bug because we have bug 422177 where we restart where the previous search left off, this isn't bad at all. We make sure we display any word boundary matches from history before trying to find non-word boundary matches.

This lets us match "toronto" for "".
Whiteboard: [has patch][need dietrich review][need bug 429498] → [has patch][need dietrich review]
Attachment #316349 - Flags: ui-review?(beltzner) → ui-review+
This is important, marking as blocking.
Flags: blocking-firefox3? → blocking-firefox3+
With this bug fixed, it will be nice to have Cyrillic letters added to the list of non-word-boundaries as mentioned in bug 393678#c33. Thanks.
Comment on attachment 316351 [details] [diff] [review]

r=me, thanks!
Attachment #316351 - Flags: review?(dietrich) → review+
Comment on attachment 316351 [details] [diff] [review]

Changes some prefs to allow showing non-word boundary matches below word boundary matches.. with testcase too!
Attachment #316351 - Flags: approval1.9?
Whiteboard: [has patch][need dietrich review] → [has patch][has reviews][has approval]
Comment on attachment 316351 [details] [diff] [review]

Attachment #316351 - Flags: approval1.9? → approval1.9+

Checking in browser/app/profile/firefox.js;
/cvsroot/mozilla/browser/app/profile/firefox.js,v  <--  firefox.js
new revision: 1.325; previous revision: 1.324
Checking in toolkit/components/places/src/nsNavHistory.cpp;
/cvsroot/mozilla/toolkit/components/places/src/nsNavHistory.cpp,v  <--  nsNavHistory.cpp
new revision: 1.297; previous revision: 1.296
Checking in toolkit/components/places/src/nsNavHistory.h;
/cvsroot/mozilla/toolkit/components/places/src/nsNavHistory.h,v  <--  nsNavHistory.h
new revision: 1.157; previous revision: 1.156
Checking in toolkit/components/places/src/nsNavHistoryAutoComplete.cpp;
/cvsroot/mozilla/toolkit/components/places/src/nsNavHistoryAutoComplete.cpp,v  <--  nsNavHistoryAutoComplete.cpp
new revision: 1.58; previous revision: 1.57
Checking in toolkit/components/places/tests/autocomplete/test_word_boundary_search.js;
/cvsroot/mozilla/toolkit/components/places/tests/autocomplete/test_word_boundary_search.js,v  <--  test_word_boundary_search.js
new revision: 1.5; previous revision: 1.4
Blocks: 405745
Closed: 16 years ago
Flags: in-testsuite+
Priority: -- → P2
Resolution: --- → FIXED
Whiteboard: [has patch][has reviews][has approval]
Target Milestone: --- → Firefox 3
Blocks: 430679
You need to log in before you can comment on or make changes to this bug.