Closed
Bug 1381716
Opened 8 years ago
Closed 8 years ago
The Onboarding tour should also detect the url of the activity-stream page for display
Categories
(Firefox :: General, enhancement, P2)
Firefox
General
Tracking
()
RESOLVED
WORKSFORME
People
(Reporter: Fischer, Unassigned)
References
Details
We should also detect if the url of the activity-stream page is present then display the onboarding tour. Currently only detect about:home and about:newtab.
Reporter | ||
Updated•8 years ago
|
Flags: qe-verify+
Priority: -- → P2
QA Contact: jwilliams
Whiteboard: [photon-onboarding]
Version: 55 Branch → unspecified
Reporter | ||
Updated•8 years ago
|
Assignee: nobody → fliu
Updated•8 years ago
|
Status: NEW → ASSIGNED
Priority: P2 → P1
Reporter | ||
Updated•8 years ago
|
Assignee: fliu → nobody
Status: ASSIGNED → NEW
Priority: P1 → P2
Comment 1•8 years ago
|
||
mardak, I guess activity-stream will still use `about:home` URL once AS replaced current about:home page, is it right?
If so we don't need to handle this issue.
Flags: needinfo?(edilee)
Comment 2•8 years ago
|
||
Yeah it should just work when activity stream replaces about:home. Just like how onboarding already works on the about:newtab page (for both activity stream and tiles).
Status: NEW → RESOLVED
Closed: 8 years ago
Flags: needinfo?(edilee)
Resolution: --- → FIXED
Updated•8 years ago
|
Resolution: FIXED → WONTFIX
Updated•8 years ago
|
Resolution: WONTFIX → WORKSFORME
Reporter | ||
Comment 4•8 years ago
|
||
Hi Kate,
Just to check with you about the schedule that put the AS page in about:home. We haven't seen that happened until now. Dose the AS team still plan to do that in FF57?
Because we are seeing some discussion about the onboarding tour on the old about:home, if the AS page will be about:home, then everything is fine. If not, the onboarding probably needs some discussion about that, thank you.
p.s Ed is PTO so ni? you.
Flags: needinfo?(khudson)
Comment 5•8 years ago
|
||
Yes the plan is to have `about:home` uri load activity stream https://github.com/mozilla/activity-stream/issues/2598
So if the current onboarding code is detecting "about:home" then it should just work.
Flags: needinfo?(khudson)
You need to log in
before you can comment on or make changes to this bug.
Description
•