On dark GTK themes the toolbar menubuttons are lighter than the other buttons

RESOLVED FIXED in Thunderbird 48.0

Status

Product:
Thunderbird
Component:
Theme
Type:
defect
Status:
RESOLVED FIXED
Opened:
3 years ago
Updated:
3 years ago

People

(Reporter: Paenglab, Assigned: Paenglab)

Tracking

Version:
unspecified
Target:
Thunderbird 48.0
Platform:
All
Linux

Thunderbird Tracking Flags

(thunderbird46 wontfix, thunderbird47 fixed, thunderbird48 fixed, thunderbird_esr4546+ fixed)

Details

Attachments

(2 attachments)

DarkThemeButton.png
5.30 KB, image/png
Details
linuxButtonFix.patch
5.22 KB, patch
mkmelin
: review+
jorgk
: approval-comm-aurora+
jorgk
: approval-comm-beta-
rkent
: approval-comm-esr45+
Details | Diff | Splinter Review
Posted image DarkThemeButton.png — 
Especially on the message headers is the difference good visible.
This happens because the semi transparent background is applied on toolbarbutton-1[type="menu-button"] and on the .toolbarbutton-menubutton-button/.toolbarbutton-menubutton-dropmarker.

On light themes this isn't visible.
Assignee: nobody → richard.marti
Status: NEW → ASSIGNED
Attachment #8739775 - Flags: review?(mkmelin+mozilla)
Comment on attachment 8739775 [details] [diff] [review]
linuxButtonFix.patch

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

Ah details:) r=mkmelin
Attachment #8739775 - Flags: review?(mkmelin+mozilla) → review+
http://hg.mozilla.org/comm-central/rev/49c8eeee2f6f
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 48.0
Comment on attachment 8739775 [details] [diff] [review]
linuxButtonFix.patch

[Approval Request Comment]
User impact if declined: not consistent toolbar button background colors on toolbars with dark desktop themes.
Testing completed (on c-c, etc.): on c-c
Risk to taking this patch (and alternatives if risky): low
Attachment #8739775 - Flags: approval-comm-esr45?
Attachment #8739775 - Flags: approval-comm-beta?
Attachment #8739775 - Flags: approval-comm-aurora?
Attachment #8739775 - Flags: approval-comm-aurora? → approval-comm-aurora+
Attachment #8739775 - Flags: approval-comm-esr45? → approval-comm-esr45+
Attachment #8739775 - Flags: approval-comm-beta? → approval-comm-beta-
You need to log in before you can comment on or make changes to this bug.