Open Bug 2056428 Opened 2 months ago Updated 2 months ago

Cleanup Download Details dialog styles Post Nova

Categories

(Firefox :: Downloads Panel, enhancement)

enhancement

Tracking

()

People

(Reporter: mhynson, Unassigned)

References

Details

(Whiteboard: [fidefe-post-nova])

Remove code for Blocked Subview Icons
With the transition to Nova, UX has removed the icons that appear within the Downloads details dialog that is available for blocked downloads. Temporarily, this was handled with display: none behind the nova pref. However, the best way to remove the dated styles once Nova is rolled out.

See - https://searchfox.org/firefox-main/rev/7fa9b602777418732e08b26d1ef6c9945c4bbd72/browser/themes/shared/downloads/download-blockedStates.css#13-27,43-50

Note that the icons still need to appear within the Downloads panel, so the rule should be modified from this:

#downloadsPanel-blockedSubview[verdict="Insecure"] .downloadsPanel-blockedSubview-image,
#downloadsPanel-blockedSubview[verdict="Malware"] .downloadsPanel-blockedSubview-image,
#downloadsListBox > richlistitem[verdict="Insecure"] .downloadBlockedBadge,
#downloadsListBox > richlistitem[verdict="Malware"] .downloadBlockedBadge {
  background-image: url("chrome://global/skin/icons/error.svg");
  -moz-context-properties: fill;
  fill: var(--icon-color-critical);
}

to this, for example:

#downloadsListBox > richlistitem[verdict="Insecure"] .downloadBlockedBadge,
#downloadsListBox > richlistitem[verdict="Malware"] .downloadBlockedBadge {
  background-image: url("chrome://global/skin/icons/error.svg");
  -moz-context-properties: fill;
  fill: var(--icon-color-critical);
}

Remove "not Nova" code in downloads.inc.css
There's a few lines of code that target the pre-Nova version of the panel and subview. This code can be removed.

Whiteboard: [fidefe-post-nova]
You need to log in before you can comment on or make changes to this bug.