Closed Bug 1381010 Opened 4 years ago Closed 4 years ago
After resizing the window, open the onboarding overlay but the tours would not appear
4.28 MB, image/gif
3.55 MB, image/gif
Bug 1381010 - After resizing the window, open the onboarding overlay but the tours would not appear.
59 bytes, text/x-review-board-request
STR: 1. Open about:home or about:newtab 2. Click the onboarding fox icon to open the onboarding overlay 3. Make sure the tours appear 4. Resize the window so that the onboarding overlay and the fox icon disappear 5. Resize the window again so that the fox icon appears again 6. Click the onboarding fox icon to open the onboarding overlay Expected result: The tours appear again. Actual result: The tours do not appear. See the attached tours_disappear_after_resizing.gif for the real action. Root Cause: We are doing lazy load of the tours , which is good. However, at the step 2, the tours got loaded into the overlay once so `_tourItems` would be loaded with tours. Then at the step 6 we thought the tours should be loaded already because `_tourItems` was not empty. But in fact, we destroyed the overlay and created one new overlay without any tour loaded.  https://dxr.mozilla.org/mozilla-central/rev/30ea2905130e85f9e1d8d56fa3097901eec6514b/browser/extensions/onboarding/content/onboarding.js#413
Assignee: nobody → fliu
Status: NEW → ASSIGNED
Priority: -- → P1
Version: 55 Branch → 56 Branch
(In reply to Fischer [:Fischer] from comment #2) > Created attachment 8886585 [details] > Bug 1381010 - After resizing the window, open the onboarding overlay but the > tours would not appear. > > Review commit: https://reviewboard.mozilla.org/r/157406/diff/#index_header > See other reviews: https://reviewboard.mozilla.org/r/157406/ Hi Mossop, Please see attachment 8886584 [details]: fix_tours_disappear_after_resizing.gif for the fix in action, thanks
Comment on attachment 8886585 [details] Bug 1381010 - After resizing the window, open the onboarding overlay but the tours would not appear. https://reviewboard.mozilla.org/r/157406/#review162566
Attachment #8886585 - Flags: review?(dtownsend) → review+
Pushed by email@example.com: https://hg.mozilla.org/integration/autoland/rev/c94795e84153 After resizing the window, open the onboarding overlay but the tours would not appear. r=mossop
I can still reproduce this bug after pull the latest Nightly. Could you confirm that?
Sorry, false alarm.
I have verified that this bug is fixed with today's nightly.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.