Closed Bug 1680432 Opened 5 years ago Closed 5 years ago

Wrong text color of folder when receiving new message and the focus moves somewhere else

Categories

(Thunderbird :: Theme, defect)

Desktop
macOS
defect

Tracking

(thunderbird_esr78? fixed, thunderbird84? affected)

RESOLVED FIXED
85 Branch
Tracking Status
thunderbird_esr78 ? fixed
thunderbird84 ? affected

People

(Reporter: aleca, Assigned: Paenglab)

Details

Attachments

(2 files)

This happens only on macOS, and it presents itself right after receiving a new message and the focus moves somewhere else.
The folder text color turns white, which doesn't really work well with the light grey background.

We don't need the HighlightText as macOS has no Highlight background treechildren when selected.

Assignee: nobody → richard.marti
Status: NEW → ASSIGNED
Attachment #9191136 - Flags: review?(alessandro)
Comment on attachment 9191136 [details] [diff] [review] 1680432-mac-folderPane-no-highlight-color.patch Review of attachment 9191136 [details] [diff] [review]: ----------------------------------------------------------------- Perfect!
Attachment #9191136 - Flags: review?(alessandro) → review+
Target Milestone: --- → 85 Branch

Pushed by geoff@darktrojan.net:
https://hg.mozilla.org/comm-central/rev/56fcc344e49e
Mac default theme: Don't use the highlight text color for selected folders with new messages. r=aleca

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

Comment on attachment 9191136 [details] [diff] [review]
1680432-mac-folderPane-no-highlight-color.patch

[Approval Request Comment]
User impact if declined: only Mac, low contrast when folder with new message is selected on default theme
Testing completed (on c-c, etc.): on c-c
Risk to taking this patch (and alternatives if risky): low

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

Comment on attachment 9191136 [details] [diff] [review]
1680432-mac-folderPane-no-highlight-color.patch

[Triage Comment]
Approved for esr78
(no beta uplift, so this will be in beta 85)

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

Attachment

General

Created:
Updated:
Size: