Closed Bug 1261926 Opened 9 years ago Closed 9 years ago

Better inclusion of bookmarks in awesomebar search results

Categories

(Firefox for iOS :: Data Storage, defect)

All
iOS
defect
Not set
normal

Tracking

()

RESOLVED FIXED
Tracking Status
fxios 6.0+ ---
fxios-v5.0 --- fixed

People

(Reporter: rnewman, Assigned: fluffyemily)

References

Details

(Whiteboard: [target 5.0])

Attachments

(1 file)

48 bytes, text/x-github-pull-request
sleroux
: review+
Details | Review
The first stab at this landed in Bug 1185038. This adds 0-5 matching bookmarks at the front of the matching history results. This is missing a few things: * You can get duplicate bookmarks if they're present multiple times in the DB (e.g., in your Toolbar and in Menu, or bookmarked locally and remotely). * You can get duplicates between bookmarks and history -- e.g., visit Reddit, bookmark it, search for "reddit", and you'll see two rows. * History items aren't marked with the star, even if they're not present in the list at the top. See Bug 1185038 for discussion of these limitations and possibilities for improving them.
Nomming for prioritization.
tracking-fxios: --- → ?
Whiteboard: [nicetohave]
Assignee: nobody → etoop
Status: NEW → ASSIGNED
Attached file Pull request
:rnewman - I added you in case you were interested, if you're too busy don't worry.
Attachment #8759647 - Flags: review?(sleroux)
Attachment #8759647 - Flags: review?(rnewman)
Comment on attachment 8759647 [details] [review] Pull request I _think_ this looks pretty good. Just left a question about the alias. The feature works pretty well on my bookmarks and from what I can tell from the SQL changes to UNION -> UNION ALL and the added JOIN it looks good but my SQL-foo isn't the greatest so I'll defer to Richard if he has any additional comments.
Attachment #8759647 - Flags: review?(sleroux) → review+
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Whiteboard: [nicetohave] → [target 5.0][needsuplift]
v5.x 05006be3a52ad7c2cc4a8ba929f0e942591845f2
Whiteboard: [target 5.0][needsuplift] → [target 5.0]
Attachment #8759647 - Flags: review?(rnewman)
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: