Closed Bug 1567433 Opened 5 years ago Closed 5 years ago

"Tracking Protections" doesn't have pressing effect

Categories

(Firefox :: Site Identity, defect, P1)

70 Branch
Desktop
All
defect

Tracking

()

VERIFIED FIXED
Firefox 70
Tracking Status
firefox-esr60 --- unaffected
firefox-esr68 --- unaffected
firefox68 --- unaffected
firefox69 --- unaffected
firefox70 --- verified

People

(Reporter: Virtual, Assigned: timhuang)

References

Details

(Keywords: nightly-community, ux-consistency, Whiteboard: [skyline])

Attachments

(8 files)

11.49 KB, image/png
Details
10.17 KB, image/png
Details
10.18 KB, image/png
Details
10.17 KB, image/png
Details
10.18 KB, image/png
Details
15.56 KB, image/png
Details
47 bytes, text/x-phabricator-request
Details | Review
47 bytes, text/x-phabricator-request
Details | Review

STR:

  1. Keep pressing "Tracking Protections"
    and see that it doesn't have pressing pressing effect like other items in bars.

It only shows itself after "Tracking Protections" menu will be completely visible.

Compare attachments from:

Using the dark theme:
Background of padlock is brighter on mouse down (and when the identity panel is open) than when hovering.
Background of TP icon is only brigther when TP panel is open. It should be brigther on mouse down, too.

OS: Windows 7 → All
Hardware: x86_64 → Desktop
Whiteboard: [privacy-panel][triage]
Priority: -- → P1
Whiteboard: [privacy-panel][triage] → [skyline]
Assignee: nobody → tihuang
Status: NEW → ASSIGNED

This patches fixes this issue by adding a missing css rule for the
pressing effect.

This patch moves the setting of 'open' attribute to a right place. We
should set it before we open the panel instead of in the event
'onpopupshown'.

Depends on D39494

Pushed by tihuang@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/2c0bc0b959a4
Part 1: Fix the issue that tracking protection icon doesn't have pressing effect. r=johannh
https://hg.mozilla.org/integration/autoland/rev/4580b16fa5ee
Part 2: Fixing the issue that hover state of the tracking protection icon is incorrect during opening the protection panel. r=johannh
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 70

I'm confirming that bug is fixed, starting in Mozilla Firefox Nightly 70.0a1 (2019-07-30), so I'm marking this bug as VERIFIED.
Thank you very much! \o/

Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: