Contact tree in AB not dark on Mac with dark theme

RESOLVED FIXED in Thunderbird 66.0

Status

defect
RESOLVED FIXED
5 months ago
5 months ago

People

(Reporter: Paenglab, Assigned: Paenglab)

Tracking

unspecified
Thunderbird 66.0
All
macOS

Thunderbird Tracking Flags

(thunderbird65 fixed, thunderbird66 fixed)

Details

Attachments

(2 attachments, 1 obsolete attachment)

(Assignee)

Description

5 months ago
With the dark theme, the contacts tree in AB is still bright but the entries are hard readable.
(Assignee)

Comment 1

5 months ago
Posted patch 1517285-AB-themed-tree.patch (obsolete) — Splinter Review
We shouldn't use the transparency on this tree at all.

As a preparation for theming the splitters I made the splitters visually only 1px.
Assignee: nobody → richard.marti
Status: NEW → ASSIGNED
Attachment #9033986 - Flags: review?(arshdkhn1)
Posted image addressbook_sc.png
everything looks good except when any contact is clicked, you can see in the screenshot that anything is hardly visible.
Attachment #9033986 - Flags: review?(arshdkhn1) → review-
(Assignee)

Comment 3

5 months ago
Card display fixed.
Attachment #9033986 - Attachment is obsolete: true
Attachment #9034369 - Flags: review?(arshdkhn1)
Comment on attachment 9034369 [details] [diff] [review]
1517285-AB-themed-tree.patch

Review of attachment 9034369 [details] [diff] [review]:
-----------------------------------------------------------------

lgtm. I am not sure what is the guideline for using dark and light background. How do you decide it?
Attachment #9034369 - Flags: review?(arshdkhn1) → review+
(Assignee)

Comment 5

5 months ago
It's depending the selected theme in Add-ons Manager. On Mac, when you use on Mac the dark theme TB, and also FX, switches to the dark theme also when the default theme is selected in Add-ons Manager.

I know there are some places, like chat, where we need some work to look correct with dark theme.
Keywords: checkin-needed

Comment 6

5 months ago
Pushed by mozilla@jorgk.com:
https://hg.mozilla.org/comm-central/rev/fc3f22843429
Fix AB contact tree on Mac for dark theme by not using -moz-appearance: -moz-mac-source-list. r=arshad
Status: ASSIGNED → RESOLVED
Last Resolved: 5 months ago
Keywords: checkin-needed
Resolution: --- → FIXED

Updated

5 months ago
Target Milestone: --- → Thunderbird 66.0

Comment 7

5 months ago
Comment on attachment 9034369 [details] [diff] [review]
1517285-AB-themed-tree.patch

I guess this will need beta uplift, right?
Attachment #9034369 - Flags: approval-comm-beta+
(Assignee)

Comment 8

5 months ago
Yes.
You need to log in before you can comment on or make changes to this bug.