Closed
Bug 1990972
Opened 2 months ago
Closed 2 hours ago
Allow IP Protection exclusions to be modified from the settings page
Categories
(Firefox :: IP Protection, task, P1)
Firefox
IP Protection
Tracking
()
RESOLVED
INVALID
People
(Reporter: kpatenio, Unassigned)
References
Details
(Whiteboard: [fx-vpn])
Attachments
(1 file)
|
99.92 KB,
image/png
|
Details |
Follow up to Bug 1990013. We want the ability to modify site exclusions from the settings page once the UI is implemented.
Things to add:
- See registered site exclusions (if any) in the settings UI
- Add new domains through the UI
- Remove existing domains from the UI
Loading the exclusions
- We can easily access the set of exceptions via
IPPExceptionsManager. With this, we can potentially pass off the domains to the custom component for the settings UI.
Possible approach to add / remove domains
Perhaps we can take a similar to approach to dohExceptions.js:
- After we load the exceptions by reading our exclusions / inclusions prefs and populate the sets, we can run logic that "builds" the list for us in the dialog. dohExceptions calls a method that does this.
- There's two options: either we expand
IPPExceptionsManagerto do the building for us, or we attach another JS file specific to the dialog's behaviour; our script will useIPPExceptionsManagerto check if the sets are populated and then iterate through them.
Figma
Updated•2 months ago
|
Update: this is also a follow up to Bug 1993334, which handles the actual rendering of the exceptions dialogs.
Depends on: 1993334
Requirements indicated in this bug are outdated, for we now rely on the Permissions system to add site exceptions. Plus, we are transitioning to the MVP version of the feature, which will have significant UI changes. Closing this bug since it's no longer relevant.
Status: NEW → RESOLVED
Closed: 2 hours ago
Resolution: --- → INVALID
You need to log in
before you can comment on or make changes to this bug.
Description
•