Closed Bug 1555226 Opened 6 months ago Closed 5 months ago

[Protections Panel] Implement toggle switch flow with mini panel

Categories

(Firefox :: Site Identity, task, P1)

task

Tracking

()

RESOLVED FIXED
Firefox 69
Tracking Status
firefox69 --- fixed

People

(Reporter: nhnt11, Assigned: timhuang)

References

Details

Attachments

(6 files)

With the current version of the patch in bug 1551902, when the toggle switch is triggered, the panel remains open and the UI changes state correctly. The specification dictates that we should wait 500ms, then refresh the page and close the panel. Once the site is done loading, we should show the mini panel indicating the new state.

Priority: -- → P1

I talked to Tim and he agreed to pick this up.

Assignee: nobody → tihuang
Status: NEW → ASSIGNED

This patch moves the 'enabled' attributes to
'protections-popup-tp-switch-section' from
'protections-popup-tp-switch'. In order to change the background as
toggling the TP switch button, there should be a single attribute to
control both background and the TP switch. So, the 'enabled' attribute
should live in the section, so that both section and TP switch can
reference this value by the CSS attribute selector.

This patch implements the toggle switch flow. It adds a strip background
to the TP switch section after toggled the switch to off. Changing the
string in the TP switch section.

Depends on D36015

This patch implements the capability for opening the protections panel
as a mini panel. The mini panel is a protections panel which only displays
the header section.

Depends on D36016

This patch makes the mini panel been shown after page reload due to
switch the tracking protection state. The mini panel will be auto hidden
after certain time, which can be configured by the pref
'browser.protections_panel.mini_panel_auto_hide_timing'. The default
value is 5 seconds.

This also implements the behavior that showing the protections panel on
clicking the mini panel.

Depends on D36017

This patch adds the badge at the side of TP switch toggle. The badge
will only show when the TP switch toggles to off.

Depends on D36018

Attachment #9074184 - Attachment description: Bug 1555226 - Part 2: Implementing the toggle switch flow for switching to OFf from On. r?nhnt11! → Bug 1555226 - Part 2: Implementing the toggle switch flow for the TP switch section. r?nhnt11!
Attachment #9074183 - Attachment description: Bug 1555226 - Part 1: Move the 'enabled' attribute into 'protections-popup-tp-switch-section' from 'protections-popup-tp-switch' r?nhnt11! → Bug 1555226 - Part 1: Toggle the 'hasException' attribute on the protections panel after toggling the TP switch. r?nhnt11!

This patch adds two tests. One for ensuring the mini panel is working
correctly. Another for check the correctness of the TP switch flow.

Pushed by tihuang@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/a64eedd73f97
Part 1: Toggle the 'hasException' attribute on the protections panel after toggling the TP switch. r=nhnt11
https://hg.mozilla.org/integration/autoland/rev/8e13f45f0d26
Part 2: Implementing the toggle switch flow for the TP switch section. r=nhnt11
https://hg.mozilla.org/integration/autoland/rev/ab2c77c1aa02
Part 3: Allow the protections panel to be opened as a mini panel. r=nhnt11
https://hg.mozilla.org/integration/autoland/rev/ad5b7db6a3c2
Part 4: Implementing the switch flow for the mini panel. r=nhnt11
https://hg.mozilla.org/integration/autoland/rev/734b19770a7e
Part 5: Add the badge alongside the TP switch toggle. r=nhnt11
https://hg.mozilla.org/integration/autoland/rev/58229ee31005
Part 6: Add tests for the mini panel and the TP switch flow. r=nhnt11
You need to log in before you can comment on or make changes to this bug.