Closed
Bug 1730017
Opened 4 years ago
Closed 4 years ago
Save to Pocket button in newtab Pocket cards
Categories
(Firefox :: Pocket, enhancement)
Firefox
Pocket
Tracking
()
VERIFIED
FIXED
94 Branch
People
(Reporter: thecount, Assigned: thecount)
References
(Blocks 2 open bugs)
Details
Attachments
(1 file)
This is going to start as an experiment, to see the impact of a more prominent save button in cards might have to saves overall.
A theory for this being people could save multiple interesting stories and read 1 instead of just reading 1, or save multiple interesting stories and then search.
| Assignee | ||
Updated•4 years ago
|
| Assignee | ||
Updated•4 years ago
|
Assignee: nobody → sdowne
| Assignee | ||
Comment 1•4 years ago
|
||
| Assignee | ||
Comment 2•4 years ago
•
|
||
To test:
- Set browser.newtabpage.activity-stream.discoverystream.saveToPocketCard.enabled to true
- Also probably want to set browser.newtabpage.activity-stream.discoverystream.compactLayout.enabled to true too.
- Open a new tab.
- Hover over a card. EXPECTED: Should see a save to pocket button in the thumbnail, centred, and the meatball menu next to it.
- Open the meatball menu. EXPECTED: Should open a menu the same as before, cept the save to Pocket item is now missing.
- Click the save to pocket button in the card. EXPECTED: Should open the Pocket signup panel.
- Sign in to Pocket.
- Click the save to pocket button in the card. EXPECTED: Should update the button status to saved, and should also now be saved in your Pocket account.
- Set browser.newtabpage.activity-stream.discoverystream.saveToPocketCard.enabled back to false.
- Open newtab.
- Menu and buttons should now be back to the normal experience.
Pushed by sdowne@getpocket.com:
https://hg.mozilla.org/integration/autoland/rev/ea872e0652de
Adding save to Pocket button to newtab Pocket cards on hover. r=fluent-reviewers,flod,gvn
Comment 4•4 years ago
|
||
| bugherder | ||
Status: NEW → RESOLVED
Closed: 4 years ago
status-firefox94:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 94 Branch
Updated•4 years ago
|
Flags: qe-verify+
Comment 5•4 years ago
•
|
||
This enhancement is implemented and working as expected(based on comment 2) on Firefox 98(20220304153049) and Nightly 99.0a1(20220307190508).
Edit: Also verified on Firefox 94(20211028161635).
The fix was tested on MacOS 11, Ubuntu 20.04 and a VM-run Win10 64bit.
Updated•4 years ago
|
Updated•4 years ago
|
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•