Closed Bug 1488846 Opened Last year Closed Last year

Pocket cta for logged out users flickers a bit before displaying the logged in version (topics) and shifting content

Categories

(Firefox :: New Tab Page, defect, P1)

64 Branch
defect

Tracking

()

VERIFIED FIXED
Firefox 64
Iteration:
64.1 - Sep 14
Tracking Status
firefox63 + verified
firefox64 --- verified

People

(Reporter: thecount, Assigned: thecount)

References

Details

Attachments

(1 file)

A followup bug found after landing https://bugzilla.mozilla.org/show_bug.cgi?id=1483655

And another small bug found by me caused by the same above ticket, found while I was fixing the other.

1. Flicker of logged in/out state while the page loads. You can see this in an attachment. Example: https://user-images.githubusercontent.com/197334/45060065-6ad37680-b06c-11e8-9341-f2a6cf17bba3.gif

2. The height of the cta was actually a few pixels larger than the topics, which potentially causes a slight shift in content as it loads. I made sure on max screen sizes the content doesn't move.

Initial pr fix here: https://github.com/mozilla/activity-stream/pull/4401
Assignee: nobody → sdowne
Iteration: --- → 64.1 (Sep 14)
Depends on: 1483655
Priority: -- → P1
https://github.com/mozilla/activity-stream/pull/4401
Blocks: 1489962
Status: NEW → RESOLVED
Closed: Last year
Resolution: --- → FIXED
Backout by btara@mozilla.com:
https://hg.mozilla.org/mozilla-central/rev/d2e41f2f964d
Backed out changeset 8dde92f89a24 for browser_asrouter_cfr.js failures. a=backout

Relanded:
https://hg.mozilla.org/mozilla-central/rev/581019e9ea70
I have verified that the Pocket explainer no longer flickers between its logged in/out states when the page is refreshed on the latest Nightly 64.0a1 (Build ID 20180920220102) on Windows 10 x64, Mac 10.13, and Arch Linux x64.
Status: RESOLVED → VERIFIED
Scott, why are you asking tracking for Firefox 63? Will you request an uplift of this patch for our last 63 beta this week? Thanks
Flags: needinfo?(sdowne)
Yeah, it is my plan to get this uplifted, but I hit a snag around moving the changes over that I need to work through.

It's related to the build process and not the code itself. Once I get it building the correct diff, it should be easy.
Flags: needinfo?(sdowne)
Comment on attachment 9015332 [details]
Bug 1488846 - Ensure pocket cta doesn't flash between logged in and out while loading, and ensure content doesn't shift when adding cta.

[Beta/Release Uplift Approval Request]

Feature/Bug causing the regression: Bug 1483655

User impact if declined: Bad user experience

Is this code covered by automated tests?: Yes

Has the fix been verified in Nightly?: Yes

Needs manual test from QE?: Yes

If yes, steps to reproduce: You can see it in action here: https://user-images.githubusercontent.com/197334/45060065-6ad37680-b06c-11e8-9341-f2a6cf17bba3.gif

Really just load the about:home page without being logged into pocket.

List of other uplifts needed: None

Risk to taking this patch: Low

Why is the change risky/not risky? (and alternatives if risky): Has automated tests, small code changes, and has been verified on nightly for some time now.

String changes made/needed: None
Attachment #9015332 - Flags: approval-mozilla-beta?
Correction, List of other uplifts needed: 1496246
Comment on attachment 9015332 [details]
Bug 1488846 - Ensure pocket cta doesn't flash between logged in and out while loading, and ensure content doesn't shift when adding cta.

On nightly for 3 weeks and verified by QA, uplift approved for 63 beta 14, thanks.
Attachment #9015332 - Flags: approval-mozilla-beta? → approval-mozilla-beta+
I have verified that this issue is no longer reproducible with the latest Firefox Beta (63.0b14 Build ID - 20181011200118) installed, on Windows 10 x64, Arch Linux and Mac 10.13.3. Now, the Pocket explainer no longer flickers between its logged in/out states when the page is refreshed.
Component: Activity Streams: Newtab → New Tab Page
You need to log in before you can comment on or make changes to this bug.