Closed Bug 1705188 Opened 4 years ago Closed 3 years ago

Get rid of browser CSS that sets popupnotification icons

Categories

(Firefox :: Theme, task, P3)

task
Points:
2

Tracking

()

RESOLVED FIXED
100 Branch
Tracking Status
firefox100 --- fixed

People

(Reporter: mconley, Assigned: bigiri)

References

(Blocks 1 open bug)

Details

(Whiteboard: [proton-cleanups])

Attachments

(1 file)

With Proton, we don't show icons in popupnotification panels unless the caller explicitly requests it at opening time. We should get rid of the old CSS rules that set the icons that we don't show anymore.

These are the ones I found at a glance:

https://searchfox.org/mozilla-central/rev/a5bf5d0720f9454687f500513ac82b0c8abce5a4/browser/themes/shared/notification-icons.inc.css#62,67,76,93-95,113-115,121-123,147,152,186-188,204-205,210,220,344,368-370,374-376,378-380,382-384,386-388,390-392,396-398,490-497,499-502

Priority: -- → P3
Points: --- → 2
Assignee: nobody → bigiri
Status: NEW → ASSIGNED

Removed CSS referencing .popup-notification-icon[...]

Pushed by bigiri@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/a78b962181ae Removed orphaned CSS for popupnotification icons r=dao
Backout by abutkovits@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/8a19793edb70 Backed out changeset a78b962181ae for causing failures at browser_all_files_referenced.js.
Attachment #9268278 - Attachment description: Bug 1705188 - Removed orphaned CSS for popupnotification icons r=Dao! → WIP: Bug 1705188 - Removed orphaned CSS for popupnotification icons r=Dao!
Attachment #9268278 - Attachment description: WIP: Bug 1705188 - Removed orphaned CSS for popupnotification icons r=Dao! → Bug 1705188 - Removed orphaned CSS for popupnotification icons r=Dao!

The images addon-install-error.svg and addon-install-warning.svg were orphaned by these changes. My new patch removes them.

Flags: needinfo?(bigiri)
Pushed by bigiri@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/10fed540fa34 Removed orphaned CSS for popupnotification icons r=dao
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → 100 Branch
Regressions: 1765453
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: