From 1618a91d2ce4ca2ad02d9f38ef094f834639f218 Mon Sep 17 00:00:00 2001 From: Alessandro Castellani Date: Tue, 18 Feb 2020 12:52:55 -0800 Subject: [PATCH] Bug 1609928 - Fix pill colour of addresses retrieved via LDAP. r=mkmelin --- mail/base/content/mailWidgets.js | 15 ------------- mail/themes/shared/mail/messengercompose.css | 22 -------------------- 2 files changed, 37 deletions(-) diff --git a/mail/base/content/mailWidgets.js b/mail/base/content/mailWidgets.js index db042cc7f0..cad660e3e7 100644 --- a/mail/base/content/mailWidgets.js +++ b/mail/base/content/mailWidgets.js @@ -1956,12 +1956,6 @@ !isValid && !isMailingList && !isNewsgroup ); - let emailCard = DisplayNameUtils.getCardForEmail(this.emailAddress); - this.classList.toggle( - "warning", - isValid && !emailCard.card && !isMailingList && !isNewsgroup - ); - this.style.removeProperty("max-width"); this.style.removeProperty("min-width"); this.classList.remove("editing"); @@ -2225,15 +2219,6 @@ !isValidAddress(address.email) && !isMailingList && !isNewsgroup ); - let emailCard = DisplayNameUtils.getCardForEmail(address.email); - pill.classList.toggle( - "warning", - isValidAddress(address.email) && - !emailCard.card && - !isMailingList && - !isNewsgroup - ); - pill.addEventListener("click", event => { this.checkSelected(pill, event); }); diff --git a/mail/themes/shared/mail/messengercompose.css b/mail/themes/shared/mail/messengercompose.css index c3ee5e1fb5..4e5a59b3ce 100644 --- a/mail/themes/shared/mail/messengercompose.css +++ b/mail/themes/shared/mail/messengercompose.css @@ -591,30 +591,8 @@ background-color: #310005; } -.address-pill.warning { - color: #a44900; - background-color: #fff2e5; -} - -.address-pill.warning:hover:not(.editing), -.address-pill.warning:focus:not(.editing) { - background-color: rgba(255,130,0,0.15); -} - -#MsgHeadersToolbar[brighttext] .address-pill.warning:not(.editing) { - color: #fe7100; - background-color: #341700; -} - -#MsgHeadersToolbar[brighttext] .address-pill.warning:hover:not(.editing), -#MsgHeadersToolbar[brighttext] .address-pill.warning:focus:not(.editing) { - background-color: #301500; -} - .address-pill[selected]:not(.editing), #MsgHeadersToolbar[brighttext] .address-pill[selected]:not(.editing), -.address-pill.warning[selected]:not(.editing), -#MsgHeadersToolbar[brighttext] .address-pill.warning[selected]:not(.editing), .address-pill.error[selected]:not(.editing), #MsgHeadersToolbar[brighttext] .address-pill.error[selected]:not(.editing) { color: HighlightText;