Implement Firefox Monitor popup notification UI

ASSIGNED
Assigned to

Status

()

ASSIGNED
6 months ago
6 months ago

People

(Reporter: nhnt11, Assigned: nhnt11)

Tracking

(Blocks: 1 bug)

62 Branch
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 2 obsolete attachments)

(Assignee)

Description

6 months ago
Created attachment 8999323 [details] [diff] [review]
WIP

In this bug my goal is to land code for showing a PopupNotification when a breached site is visited.

That includes:
- Code for observing windows and injecting popup notification UI and styles
- Code for triggering a popup notification when needed
- L10n support
- Code to keep this pref'd off

I do NOT plan to cover in THIS bug:
- Fetching actual breach data from HIBP
(Assignee)

Updated

6 months ago
Assignee: nobody → nhnt11
Status: NEW → ASSIGNED
(Assignee)

Comment 1

6 months ago
Created attachment 8999803 [details] [diff] [review]
Patch

One thing not mentioned in comment 0 is that this patch does not add telemetry yet. I plan to do that in a follow-up.

There are some TODO comments in the code, but they are either not going to be covered in this bug or are minor - so I didn't see any reason not to start getting this reviewed. I've also asked UX a few questions so I may be updating this patch when they respond - but I didn't want to wait for that to get this up.

Try push: https://treeherder.mozilla.org/#/jobs?repo=try&revision=ac645faf074f9ff11ac882ce0761f056fa4c4ed1

I will request r? once I have a green try run.
Attachment #8999323 - Attachment is obsolete: true
(Assignee)

Comment 2

6 months ago
Created attachment 8999807 [details] [diff] [review]
Patch

Previous patch was broken, fixed and added some improvements.

New try push: https://treeherder.mozilla.org/#/jobs?repo=try&revision=a196f6ec33c28ce99bced7e00ef32af67a5f8d58
Attachment #8999803 - Attachment is obsolete: true
You need to log in before you can comment on or make changes to this bug.