Closed Bug 1808441 Opened 2 years ago Closed 2 years ago

Add Cookie Banner Promo on PB new tab

Categories

(Firefox :: Messaging System, task, P1)

task

Tracking

()

VERIFIED FIXED
112 Branch
Iteration:
112.1 - Feb 13 - Feb 24
Tracking Status
firefox112 --- verified

People

(Reporter: aminomancer, Assigned: aminomancer)

References

(Blocks 2 open bugs)

Details

Attachments

(4 files)

Description

As a private window user on desktop, I want to be informed of the availability of the new cookie banner management feature so I can decide to opt in.

Acceptance Criteria

On first launch of a private window following the roll-out, users who have not opted into cookie banner handling yet are exposed to a UI informing them about the availability of the feature. 

  • The UI lets users enable cookie banner handling
  • The message gets displayed a maximum of X times
  • Telemetry helps count users prompted and engagement
  • Priority of message vs Firefox Focus, Pin Private Browsing Window and VPN
  • All locales, all regions

Figma prototype

Blocks: 1808611
Attachment #9310601 - Attachment description: Bug 1808441 - Add Cookie Banner Promo on PB new tab. r=#omc-reviewers → Bug 1808441 - Add Cookie Banner Promo on PB new tab. r=pdahiya,#omc-reviewers
Attached file Technical Plan
Attached file Pre-release Checklist
Attachment #9310601 - Attachment description: Bug 1808441 - Add Cookie Banner Promo on PB new tab. r=pdahiya,#omc-reviewers → WIP: Bug 1808441 - Add Cookie Banner Promo on PB new tab. r=pdahiya,#omc-reviewers

Hi Romain, I figured you probably have this covered already but just want to check if there will be an update to the copy for this promo. The prototype currently says "Firefox now tries to automatically reject cookies. If a reject option isn’t available, Firefox may accept all cookies to dismiss the banner." But with the new behavior after the privacy review (reject all, do nothing if banner can't be rejected), the disclaimer about possibly accepting cookies is no longer necessary/accurate. Thanks!

Edit: Don't worry about it, I found your doc and I can see the descriptions have been updated. Just give me a go ahead when the copy is final? Thanks

Flags: needinfo?(rtestard)
Flags: needinfo?(rtestard)
Iteration: --- → 110.3 - Jan 9 - Jan 13
Priority: -- → P1
Attachment #9310601 - Attachment description: WIP: Bug 1808441 - Add Cookie Banner Promo on PB new tab. r=pdahiya,#omc-reviewers → Bug 1808441 - Add Cookie Banner Promo on PB new tab. r=pdahiya,#omc-reviewers
Iteration: 110.3 - Jan 9 - Jan 13 → 111.1 - Jan 16 - Jan 27
Iteration: 111.1 - Jan 16 - Jan 27 → 111.2 - Jan 30 - Feb 10
Iteration: 111.2 - Jan 30 - Feb 10 → 112.1 - Feb 13 - Feb 24

Hi Romain, just checking to confirm that the above looks right to you. (Figma prototype)

At a glance, there's a visual discrepancy between the implementation and the prototype: the promo and info cards are wider than the searchbar, whereas in the prototype, they are of equal width. I'm thinking equal width looks better, but since that's a general issue not specific to the promo card, it's probably better resolved in its own bug in Firefox::New Tab Page.

Also, the "Learn more" link is laid out vertically in our implementation, but inline in the prototype. I'm guessing it's just a CSS display issue. And finally, the space between the searchbar and the promo card, and the promo card and the info card, are inconsistent. I can file new bugs for those issues though.

Flags: needinfo?(rtestard)

Also, the "Learn more" link is laid out vertically in our implementation, but inline in the prototype. I'm guessing it's just a CSS display issue. And finally, the space between the searchbar and the promo card, and the promo card and the info card, are inconsistent. I can file new bugs for those issues though.

Will be great to get feedback on Cookie banner specific final content and image. Promo and info (Learn More) section uses generic styles and layout and unless a blocker should be kept as is.

Pushed by shughes@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/bda8206cac2f Add Cookie Banner Promo on PB new tab. r=omc-reviewers,flod,pdahiya,emcminn
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 112 Branch

(In reply to Shane Hughes [:aminomancer] from comment #6)

Hi Romain, just checking to confirm that the above looks right to you. (Figma prototype)

At a glance, there's a visual discrepancy between the implementation and the prototype: the promo and info cards are wider than the searchbar, whereas in the prototype, they are of equal width. I'm thinking equal width looks better, but since that's a general issue not specific to the promo card, it's probably better resolved in its own bug in Firefox::New Tab Page.

Also, the "Learn more" link is laid out vertically in our implementation, but inline in the prototype. I'm guessing it's just a CSS display issue. And finally, the space between the searchbar and the promo card, and the promo card and the info card, are inconsistent. I can file new bugs for those issues though.

Thanks Shane

  • Regarding the "Learn more" link - I think in line is right given it's the current set-up on private new tab @misha can confirm
  • Regarding the promo and info cards width - your implementation seems to align with the current private new tab set-up so I wonder if the width on Figma was intentional - @misha?
  • I notice the call to action button color is different - @misha can you please confim which is the right one?
  • the
Flags: needinfo?(rtestard) → needinfo?(mbruk)

Ah, nice catch. Thanks for pointing that out... the call to action button color is a bit of an oddity. Normally, private browsing windows get a dark theme, regardless of the user's settings. But the way that's handled is different from the actual dark theme. So, if you have light mode enabled and open a PBM window, some things will still be colored like they are in a normal light mode window. I'm sure this is a bug, since designs for private browsing seem to consistently use the dark color scheme, and everything else in PBM uses it. I'll file a bug.

See Also: → 1817882

I have verified this task using the steps provided here and I can confirm the following:

  • The "Cookie Banner Promo" is displayed on the "Private Browsing" window if the browser.promo.cookiebanners.enabled pref is set to true in the "about:config" page.
  • The cookiebanners.service.mode pref is set to 1 after the primary button from the "Cookie Banner Promo" is clicked.
  • The "Cookie Banner Promo" is no longer displayed on new "Private Browsing" windows after the primary button was clicked.

Verified using the latest Firefox Nightly (112.0a1 Build ID - 20230221052246) installed on Windows 10 x64, macOS 12.6.1, and Ubuntu 22.04 x64.

Status: RESOLVED → VERIFIED

(In reply to Romain Testard [:RT] from comment #10)

(In reply to Shane Hughes [:aminomancer] from comment #6)

Hi Romain, just checking to confirm that the above looks right to you. (Figma prototype)

At a glance, there's a visual discrepancy between the implementation and the prototype: the promo and info cards are wider than the searchbar, whereas in the prototype, they are of equal width. I'm thinking equal width looks better, but since that's a general issue not specific to the promo card, it's probably better resolved in its own bug in Firefox::New Tab Page.

Also, the "Learn more" link is laid out vertically in our implementation, but inline in the prototype. I'm guessing it's just a CSS display issue. And finally, the space between the searchbar and the promo card, and the promo card and the info card, are inconsistent. I can file new bugs for those issues though.

Thanks Shane

  • Regarding the "Learn more" link - I think in line is right given it's the current set-up on private new tab @misha can confirm
  • Regarding the promo and info cards width - your implementation seems to align with the current private new tab set-up so I wonder if the width on Figma was intentional - @misha?
  • I notice the call to action button color is different - @misha can you please confim which is the right one?
  • the

Thanks for catching this!
Regarding the "Learn more" link - in line is right :)

  • Regarding the promo and info cards width - your implementation looks good
  • I notice the call to action button color is different - the button should be our dark theme color!
Flags: needinfo?(mbruk)

(In reply to Shane Hughes [:aminomancer] from comment #6)

Hi Romain, just checking to confirm that the above looks right to you. (Figma prototype)

At a glance, there's a visual discrepancy between the implementation and the prototype: the promo and info cards are wider than the searchbar, whereas in the prototype, they are of equal width. I'm thinking equal width looks better, but since that's a general issue not specific to the promo card, it's probably better resolved in its own bug in Firefox::New Tab Page.

Also, the "Learn more" link is laid out vertically in our implementation, but inline in the prototype. I'm guessing it's just a CSS display issue. And finally, the space between the searchbar and the promo card, and the promo card and the info card, are inconsistent. I can file new bugs for those issues though.

not sure if this was already addressed but the illustration looks broken (the stop sign stroke, let me know if i can assist in exporting the graphic!)

Regressions: 1818345
Blocks: 1821818
Blocks: 1855815
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: