Implement Firefox Monitor popup notification UI

RESOLVED FIXED

Status

()

enhancement
RESOLVED FIXED
10 months ago
22 days ago

People

(Reporter: nhnt11, Assigned: nhnt11)

Tracking

(Blocks 1 bug)

62 Branch
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment, 2 obsolete attachments)

Assignee

Description

10 months ago
Posted patch WIP (obsolete) — Splinter Review
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

10 months ago
Assignee: nobody → nhnt11
Status: NEW → ASSIGNED
Assignee

Comment 1

10 months ago
Posted patch Patch (obsolete) — Splinter Review
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

10 months ago
Posted patch PatchSplinter Review
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
Assignee

Comment 3

22 days ago

This was implemented elsewhere and now lives as a system add-on and is pref'd on after bug 1531838.

Status: ASSIGNED → RESOLVED
Last Resolved: 22 days ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.