Save to Pocket refresh prefs and nimbus implementation
Categories
(Firefox :: Pocket, enhancement)
Tracking
()
Tracking | Status | |
---|---|---|
firefox97 | --- | fixed |
People
(Reporter: thecount, Assigned: thecount)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
Assignee | ||
Updated•4 years ago
|
Assignee | ||
Comment 1•4 years ago
|
||
Assignee | ||
Comment 2•4 years ago
|
||
To test:
You'll need to open and verify each panel, then flip some prefs, and verify each panel again.
To open a panel and verify their query params, do this:
- Enable "Disable Popup Auto-Hide" from multiprocess toolbox.
- Click the save to Pocket button.
- Because "Disable Popup Auto-Hide" is set, the panel should stay open, allowing you to inspect it. You can do this with the "Pick an element from the page" button in the top left of the multiprocess toolbox.
- in the inspector, find the url in the browser element, and check the query params.
You want to do steps 1-4 for each panel, signup, saved, and home (click the Pocket button from a newtab to get the home panel).
You need to toggle "Disable Popup Auto-Hide" on and off to open and close panels as needed.
Go about about:config and flip extensions.pocket.refresh.layout.enabled and extensions.pocket.refresh.emailButton.enabled to true
Now test each panel with the prefs flipped to true.
Expected:
Signup panel should have search params: utmCampaign, utmSource, emailButton, layoutRefresh, pockethost, and locale
Home panel should have search params: layoutRefresh, pockethost, and locale
Saved panel should have search params: premiumStatus, fxasignedin, layoutRefresh, pockethost, and locale
emailButton, layoutRefresh should flip based on the prefs. No other search params should be changing from what's on master.
Comment 4•4 years ago
|
||
bugherder |
Description
•