Closed Bug 1668410 Opened 4 years ago Closed 4 years ago

Fix inline notifications background color on Dark mode

Categories

(Thunderbird :: Theme, defect, P1)

Tracking

(thunderbird_esr68 unaffected, thunderbird_esr78+ fixed, thunderbird81 affected, thunderbird82 fixed, thunderbird83 affected)

RESOLVED FIXED
83 Branch
Tracking Status
thunderbird_esr68 --- unaffected
thunderbird_esr78 + fixed
thunderbird81 --- affected
thunderbird82 --- fixed
thunderbird83 --- affected

People

(Reporter: aleca, Assigned: aleca)

Details

Attachments

(2 files)

Attached image Security pane.png

The inlineNotification.css file comes with hard-coded background colours for error and success variations.
These colours don't play well with the Dark theme, making the whole notification completely unreadable.
My bad, fix coming through.

Attachment #9178879 - Flags: review?(richard.marti)

Comment on attachment 9178879 [details] [diff] [review]
1668410-inline-notifications.diff

Yes, a lower specificity for the color rule is better.

Attachment #9178879 - Flags: review?(richard.marti) → review+
Target Milestone: --- → 83 Branch
Summary: Fix inline notifications background color on Dark mdoe → Fix inline notifications background color on Dark mode

Pushed by geoff@darktrojan.net:
https://hg.mozilla.org/comm-central/rev/f4f6378c4071
Fix inline notification background colors for dark mode. r=Paenglab

Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED

Comment on attachment 9178879 [details] [diff] [review]
1668410-inline-notifications.diff

[Approval Request Comment]
Safe openpgp darkmode polish

Attachment #9178879 - Flags: approval-comm-esr78?
Attachment #9178879 - Flags: approval-comm-beta?

Comment on attachment 9178879 [details] [diff] [review]
1668410-inline-notifications.diff

[Triage Comment]
Approved for beta

Attachment #9178879 - Flags: approval-comm-beta? → approval-comm-beta+

Comment on attachment 9178879 [details] [diff] [review]
1668410-inline-notifications.diff

[Triage Comment]
Approved for esr78

Attachment #9178879 - Flags: approval-comm-esr78? → approval-comm-esr78+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: