Closed Bug 1745244 Opened 3 years ago Closed 3 years ago

Message bar buttons don't use the correct colours with HCT

Categories

(Thunderbird :: Theme, task)

Tracking

(thunderbird_esr91? fixed, thunderbird96? fixed)

RESOLVED FIXED
97 Branch
Tracking Status
thunderbird_esr91 ? fixed
thunderbird96 ? fixed

People

(Reporter: Paenglab, Assigned: Paenglab)

References

Details

Attachments

(2 files, 1 obsolete file)

In Windows' high contrast mode the buttons on the message bars (content blocking etc.) use still the default theme colours and not the high contrast colours.

Attached patch 1745244-message-bar-HCT.patch (obsolete) — Splinter Review

We have to add @media (prefers-contrast) {} colour values.

The IMIP bar needs also a fix to use the correct background colour.

Assignee: nobody → richard.marti
Status: NEW → ASSIGNED
Attachment #9254588 - Flags: review?(alessandro)
Comment on attachment 9254588 [details] [diff] [review] 1745244-message-bar-HCT.patch Review of attachment 9254588 [details] [diff] [review]: ----------------------------------------------------------------- Looks good, thanks. Please update the commit message with "[Windows] Fix high contrast colors for IMIP bar buttons"
Attachment #9254588 - Flags: review?(alessandro) → review+

Updated the commit message.

Attachment #9254588 - Attachment is obsolete: true
Attachment #9254883 - Flags: review+
Target Milestone: --- → 97 Branch

Pushed by mkmelin@iki.fi:
https://hg.mozilla.org/comm-central/rev/3d340c1f9f3e
Fix high contrast colors for IMIP bar buttons. r=aleca

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

Comment on attachment 9254883 [details] [diff] [review]
1745244-message-bar-HCT.patch

[Approval Request Comment]
User impact if declined: in HCT mode unreadable "unread" messages in thread tree
Testing completed (on c-c, etc.): on c-c
Risk to taking this patch (and alternatives if risky): low

Attachment #9254883 - Flags: approval-comm-beta?

[Approval Request Comment]
User impact if declined: in HCT mode unreadable "unread" messages in thread tree
Testing completed (on c-c, etc.): on c-c
Risk to taking this patch (and alternatives if risky): low

Attachment #9255027 - Flags: approval-comm-esr91?

Comment on attachment 9254883 [details] [diff] [review]
1745244-message-bar-HCT.patch

[Triage Comment]
Approved for 96.0b2

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

Comment on attachment 9255027 [details] [diff] [review]
1745244-message-bar-HCT-ESR.patch

[Triage Comment]
Approved for esr91

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

Attachment

General

Created:
Updated:
Size: