Closed Bug 1582763 Opened 2 months ago Closed 2 months ago

[Protections Panel] Remove blue badge from shield and toogle when Tracking Protection is off

Categories

(Firefox :: Site Identity, task, P1)

task

Tracking

()

VERIFIED FIXED
Firefox 71
Tracking Status
firefox70 --- verified
firefox71 --- verified

People

(Reporter: epang, Assigned: nhnt11)

References

(Blocks 1 open bug)

Details

(Whiteboard: [privacy-panel][skyline])

Attachments

(2 files)

Remove the blue badge on the shield and toggle when Tracking Protection is turned off. The crossed-out shield is a strong enough indicator of the state.

Priority: -- → P1
Assignee: nobody → nhnt11
Status: NEW → ASSIGNED
Pushed by nhnt11@gmail.com:
https://hg.mozilla.org/integration/autoland/rev/7557fa807f7a
[Protections Panel] Remove blue badge from shield and toggle when TP is disabled. r=johannh

Comment on attachment 9094229 [details]
Bug 1582763 - [Protections Panel] Remove blue badge from shield and toggle when TP is disabled.

Beta/Release Uplift Approval Request

  • User impact if declined: Confusing/annoying blue notification-dot on the shield icon that cannot be cleared when TP is disabled for a site.
  • Is this code covered by automated tests?: No
  • Has the fix been verified in Nightly?: No
  • Needs manual test from QE?: No
  • If yes, steps to reproduce: Super simple - in the protections panel, toggle the tracking protection switch OFF for a site. Without the patch, a persistent blue dot is shown on top of the shield icon. With the patch, it is no longer present.
  • List of other uplifts needed: None
  • Risk to taking this patch: Low
  • Why is the change risky/not risky? (and alternatives if risky): Mostly CSS removal, one tiny piece of code also removed. Very low risk.
  • String changes made/needed: None
Attachment #9094229 - Flags: approval-mozilla-beta?
Status: ASSIGNED → RESOLVED
Closed: 2 months ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 71

that's fast. thank you, nihanth!

The patch looks safe but given that it just landed and has no automatic tests, I'd like to have QA verify the fix on Nightly before evaluating it for uplifting.

Flags: qe-verify+

Verified as fixed on latest Nightly 71.0a1 (2019-09-22), under macOS 10.13, Ubuntu 18.04 x64 and Windows 10 x64. The blue badge is not displayed anymore while TP is OFF, in privacy panel.

Status: RESOLVED → VERIFIED
QA Whiteboard: [qa-triaged]

Comment on attachment 9094229 [details]
Bug 1582763 - [Protections Panel] Remove blue badge from shield and toggle when TP is disabled.

Fix for skyline issue, verified in Nightly, let's uplift for beta 10.

Attachment #9094229 - Flags: approval-mozilla-beta? → approval-mozilla-beta+

I verified the fix on Firefox 70.0b9 on Windows 10 x64, macOS 10.13 and Ubuntu 18.04 x64. The bug is not reproducing anymore.

Flags: qe-verify+
You need to log in before you can comment on or make changes to this bug.