Contact tree in AB not dark on Mac with dark theme

RESOLVED FIXED in Thunderbird 66.0

Status

Product:
Thunderbird
Component:
Theme
Type:
defect
Status:
RESOLVED FIXED
Reported:
5 months ago
Modified:
5 months ago

People

(Reporter: Paenglab, Assigned: Paenglab)

Tracking

Version:
unspecified
Target:
Thunderbird 66.0
Platform:
All
macOS

Thunderbird Tracking Flags

(thunderbird65 fixed, thunderbird66 fixed)

Details

Attachments

(2 attachments, 1 obsolete attachment)

addressbook_sc.png
586.75 KB, image/png
Details
1517285-AB-themed-tree.patch
3.00 KB, patch
arshad
: review+
jorgk
: approval-comm-beta+
Details | Diff | Splinter Review
(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)

Comment 2

•
5 months ago
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 4

•
5 months ago 
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.