Closed
Bug 1488933
Opened 6 years ago
Closed 6 years ago
Link to the new content blocking onboarding from about:privatebrowsing when the CB UI is enabled
Categories
(Firefox :: Private Browsing, enhancement, P1)
Firefox
Private Browsing
Tracking
()
RESOLVED
FIXED
Firefox 64
People
(Reporter: johannh, Assigned: johannh)
References
(Blocks 1 open bug)
Details
Attachments
(1 file)
46 bytes,
text/x-phabricator-request
|
Gijs
:
review+
pascalc
:
approval-mozilla-beta+
|
Details | Review |
We have a different onboarding for CB and we need to make sure to link to it from about:privatebrowsing.
Assignee | ||
Comment 1•6 years ago
|
||
Comment 2•6 years ago
|
||
Comment on attachment 9006716 [details] Bug 1488933 - Link to the new content blocking onboarding from about:privatebrowsing when the CB UI is enabled. r=Gijs :Gijs (he/him) has approved the revision.
Attachment #9006716 -
Flags: review+
Pushed by jhofmann@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/4e57c8ca8d1b Link to the new content blocking onboarding from about:privatebrowsing when the CB UI is enabled. r=Gijs
Comment 4•6 years ago
|
||
This needs a backport, right?
Assignee | ||
Comment 5•6 years ago
|
||
Comment on attachment 9006716 [details] Bug 1488933 - Link to the new content blocking onboarding from about:privatebrowsing when the CB UI is enabled. r=Gijs Approval Request Comment [Feature/Bug causing the regression]: UI Tour changes in bug 1484585 [User impact if declined]: Users won't get the correct UI Tour for the new Content Blocking feature if they visit it from about:privatebrowsing. [Is this code covered by automated tests?]: Yes [Has the fix been verified in Nightly?]: Not yet [Needs manual test from QE? If yes, steps to reproduce]: Hard to test right now, since the backend is not in place yet. But it's a very simple patch so it shouldn't be necessary. [List of other uplifts needed for the feature/fix]: None [Is the change risky?]: No [Why is the change risky/not risky?]: Very simple patch that only updates a link on about:privatebrowsing [String changes made/needed]: None
Attachment #9006716 -
Flags: approval-mozilla-beta?
Comment 6•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/4e57c8ca8d1b
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
status-firefox64:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → Firefox 64
Comment 7•6 years ago
|
||
Comment on attachment 9006716 [details] Bug 1488933 - Link to the new content blocking onboarding from about:privatebrowsing when the CB UI is enabled. r=Gijs Uplift approved for 63 beta 5
Attachment #9006716 -
Flags: approval-mozilla-beta? → approval-mozilla-beta+
Comment 8•6 years ago
|
||
bugherder uplift |
https://hg.mozilla.org/releases/mozilla-beta/rev/245f57954951
status-firefox63:
--- → fixed
You need to log in
before you can comment on or make changes to this bug.
Description
•