In multimessage view, toolbar button text is white

RESOLVED FIXED in Thunderbird 65.0

Status

defect
RESOLVED FIXED
9 months ago
8 months ago

People

(Reporter: darktrojan, Assigned: darktrojan)

Tracking

unspecified
Thunderbird 65.0

Thunderbird Tracking Flags

(thunderbird64 fixed, thunderbird65 fixed)

Details

Attachments

(2 attachments, 2 obsolete attachments)

No description provided.
Looks like your last fix in bug 1419133 was defeated by bug 1493280.
Assignee: nobody → geoff
Status: NEW → ASSIGNED
Attachment #9019179 - Flags: review?(richard.marti)
Comment on attachment 9019179 [details] [diff] [review]
1501101-multimessage-toolbar-1.diff

Please can you do this on all platforms? The problem exists on all.
Attachment #9019179 - Flags: review?(richard.marti)
As you wish.
Attachment #9019179 - Attachment is obsolete: true
Attachment #9019522 - Flags: review?(richard.marti)
Comment on attachment 9019522 [details] [diff] [review]
1501101-multimessage-toolbar-2.diff

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

Thanks!

::: mail/themes/windows/mail/multimessageview.css
@@ +33,5 @@
>    background-image: url("chrome://messenger/skin/icons/flag-col.png");
>  }
>  
>  .toolbarbutton-1.msgHeaderView-button {
> +  color: -moz-dialogtext !important;

Not needed, but when you land it, could you change the casing to -moz-DialogText?
Attachment #9019522 - Flags: review?(richard.marti) → review+
Keywords: checkin-needed
Pushed by mozilla@jorgk.com:
https://hg.mozilla.org/comm-central/rev/af375c660457
Fix text contrast on buttons in multimessageview's header. r=Paenglab
Status: ASSIGNED → RESOLVED
Closed: 9 months ago
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 65.0
Comment on attachment 9019613 [details] [diff] [review]
1501101-multimessage-toolbar-3.diff

Worth uplifting, if we're going to have another 64 beta.
Attachment #9019613 - Flags: approval-comm-beta?
Had I known earlier, it would be in the current beta built on the 26th.
Attachment #9019613 - Flags: approval-comm-beta? → approval-comm-beta+
Attachment #9019522 - Attachment is obsolete: true
You need to log in before you can comment on or make changes to this bug.