Closed
Bug 1841447
Opened 1 year ago
Closed 1 year ago
Change the "Top pick" label to "Recommended reads" for Pocket suggestions
Categories
(Firefox :: Address Bar, task, P1)
Firefox
Address Bar
Tracking
()
VERIFIED
FIXED
117 Branch
People
(Reporter: adw, Assigned: adw)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
(deleted),
text/x-phabricator-request
|
diannaS
:
approval-mozilla-beta+
|
Details |
No description provided.
Assignee | ||
Comment 1•1 year ago
|
||
Depends on D182634
Pushed by dwillcoxon@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/b87c580577d0
Change the "Top pick" label to "Recommended reads" for Pocket suggestions. r=daisuke,fluent-reviewers,flod
Assignee | ||
Comment 3•1 year ago
|
||
STR for QA:
- Set
browser.urlbar.pocket.featureGate
= true to enable Pocket suggestions andbrowser.urlbar.bestMatch.enabled
to enable best match - Type "breakfast" and verify a low confidence (non-top-pick) Pocket suggestion appears
- Verify the label above the suggestion is "Firefox Suggest"
- Type "breakfast paninis" and verify a high confidence (top-pick) Pocket suggestion appears
- Verify the label above the suggestion is "Recommended reads"
Flags: qe-verify+
Flags: in-testsuite+
Assignee | ||
Comment 4•1 year ago
|
||
Comment on attachment 9342055 [details]
Bug 1841447 - Change the "Top pick" label to "Recommended reads" for Pocket suggestions.
Beta/Release Uplift Approval Request
- User impact if declined: This bug is necessary for the Pocket suggestions feature we intend to ship as an experiment in 116.
- Is this code covered by automated tests?: Yes
- Has the fix been verified in Nightly?: No
- Needs manual test from QE?: Yes
- If yes, steps to reproduce: Please see STR in previous comment
- List of other uplifts needed: The following bugs must be uplifted first: 1841354, Bug 1841295, Bug 1841553, Bug 1841408, Bug 1841442, Bug 1837097, Bug 1841446
- Risk to taking this patch: Low
- Why is the change risky/not risky? (and alternatives if risky): Minor string change to Pocket suggestions, which are disabled by default. Has tests.
Note: This patch modifies the Firefox Suggest FTL file, but the file is not localized. It is en-US only.
- String changes made/needed: None: This patch modifies the Firefox Suggest FTL file, but the file is not localized. It is en-US only.
- Is Android affected?: No
Attachment #9342055 -
Flags: approval-mozilla-beta?
Comment 5•1 year ago
|
||
bugherder |
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
status-firefox117:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 117 Branch
Updated•1 year ago
|
QA Whiteboard: [qa-triaged]
Comment 6•1 year ago
|
||
Comment on attachment 9342055 [details]
Bug 1841447 - Change the "Top pick" label to "Recommended reads" for Pocket suggestions.
Approved for 116.0b2
Attachment #9342055 -
Flags: approval-mozilla-beta? → approval-mozilla-beta+
Comment 7•1 year ago
|
||
I have verified this issue on the latest Firefox Nightly 117.0a1 (Build ID: 20230706092222), on Windows 10 x64, macOS 13.4.1, and Ubuntu 20.04 x64.
- The "Cat Cora" Pocket suggestion appears at the bottom of the Urlbar panel after typing the complete string (“breakfast”). The label displayed above the suggestion is "Firefox Suggest".
- The "Your New Favorite Breakfast Sandwich is Anything but Basic" Pocket suggestion appears at the top of the Urlbar panel after typing the full keyword (“breakfast paninis”). The label displayed above the suggestion is "Recommended reads".
Comment 8•1 year ago
|
||
bugherder uplift |
status-firefox116:
--- → fixed
Comment 9•1 year ago
|
||
I have verified this issue on the latest Firefox Beta 116.0b2 (Build ID: 20230706193912), on Windows 10 x64, macOS 13.4.1, and Ubuntu 20.04 x64.
- The "Cat Cora" Pocket suggestion appears at the bottom of the Urlbar panel after typing the complete string (“breakfast”). The label displayed above the suggestion is "Firefox Suggest".
- The "Your New Favorite Breakfast Sandwich is Anything but Basic" Pocket suggestion appears at the top of the Urlbar after typing the full keyword (“breakfast paninis”). The label displayed above the suggestion is "Recommended reads".
You need to log in
before you can comment on or make changes to this bug.
Description
•