Add "Where to use VPN" controls to IP Protection settings section
Categories
(Firefox :: IP Protection, task)
Tracking
()
| Tracking | Status | |
|---|---|---|
| firefox146 | --- | fixed |
People
(Reporter: kpatenio, Assigned: kpatenio)
References
(Blocks 1 open bug)
Details
(Whiteboard: [fx-vpn])
Attachments
(2 files)
We want a subsection in the IP Protection section to choose between "all" and "select" mode. Enabling "all" will enable site exclusions, whereas "select" will enable site inclusions.
Radio buttons
The currently selected radio button should be based on the pref browser.ipProtection.exceptionsMode, which is either "all" or "select". If somehow, the pref value is invalid, let's opt for "all" as the fallback since it is the default mode.
CTA button
We want a CTA button that eventually opens the inclusions or exclusions modal, depending on the currently selected mode. For this bug, we can simply add the button. Behaviour for actually showing the modal will be done in a follow up.
ipprotection-settings.mjs
We may want to consider delegating a custom lit component that renders all our controls (except for the header). If we end up implementing an ipprotection-settings component in Bug 1993052, then we will want to add the buttons to this component. I'm envisioning the component to have a responsive property that tracks the current protection mode (eg. this._protectionMode) and later support an @click callback for the CTA button to open a modal.
Updated•1 month ago
|
Actually, let's try implementing the "newer" layout based on the settings redesign work: https://www.figma.com/design/l75lLbgf95PuVqNafzdvel/VPN---Firefox-Integration--Pilot?node-id=12704-119907&p=f&m=dev.
Updated•1 month ago
|
Updated•23 days ago
|
Description
•