Sort top sites by frecency

RESOLVED FIXED

Status

()

RESOLVED FIXED
4 years ago
4 years ago

People

(Reporter: wesj, Unassigned)

Tracking

unspecified
x86
macOS

Firefox Tracking Flags

(fennec+)

Details

Attachments

(1 attachment)

47 bytes, text/x-github-pull-request
bnicholson
: review+
Details | Review | Splinter Review
(Reporter)

Description

4 years ago
Lets just port the Android frecency model to top sites for now.
(Reporter)

Comment 1

4 years ago
Created attachment 8584783 [details] [review]
Pull request

This is basically the Android algorythm [1]. Note, we don't include bookmarks yet here, so that bit isn't included.

[1] http://mxr.mozilla.org/mozilla-central/source/mobile/android/base/db/BrowserContract.java#55
Attachment #8584783 - Flags: review?(bnicholson)

Updated

4 years ago
Summary: Sort top sites by frecency → Sort top sites by frequency
Comment on attachment 8584783 [details] [review]
Pull request

I'm a little surprised this patch is so short, but LGTM if it works!
Attachment #8584783 - Flags: review?(bnicholson) → review+
Reverting bug title change. Note frequency + recency = frecency :)
Summary: Sort top sites by frequency → Sort top sites by frecency
tracking-fennec: --- → ?
(Reporter)

Updated

4 years ago
Status: NEW → RESOLVED
tracking-fennec: ? → +
Last Resolved: 4 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.