Bug 1491535 - Fix the warning icons after removal of Warning.png in bug 1457781. r=jorgk,MakeMyDay
authorRichard Marti <richard.marti@gmail.com>
Fri, 14 Sep 2018 09:35:53 +0200
changeset 33168 b027417cfd9fabc5e60db7e57b7d3047a0b99092
parent 33167 d197d42b67d83e5991554fa300ddeff7d1be5203
child 33169 47c06b0b47c3147f355054507bcc1001f922b1a8
push id387
push userclokep@gmail.com
push dateMon, 10 Dec 2018 21:30:47 +0000
reviewersjorgk, MakeMyDay
bugs1491535, 1457781
Bug 1491535 - Fix the warning icons after removal of Warning.png in bug 1457781. r=jorgk,MakeMyDay
calendar/base/themes/common/calendar-alarms.css
mail/components/cloudfile/content/addAccountDialog.xul
mail/components/im/themes/imAccounts.css
mail/themes/linux/mail/accountCreation.css
mail/themes/osx/mail/accountCreation.css
mail/themes/windows/jar.mn
mail/themes/windows/mail/accountCreation.css
mail/themes/windows/mail/activity/activity.css
mail/themes/windows/mail/activity/warning.png
mailnews/base/content/newsError.xhtml
--- a/calendar/base/themes/common/calendar-alarms.css
+++ b/calendar/base/themes/common/calendar-alarms.css
@@ -51,22 +51,22 @@
 }
 
 #reminder-notifications {
     overflow-y: visible;
 }
 
 #reminder-notifications > notification {
     background-color: transparent;
+    border-width: 0;
 }
-#reminder-notifications > notification > .notification-inner {
-    border: 0;
-}
-#reminder-notifications > notification[type="warning"] {
-    list-style-image: url(chrome://global/skin/icons/Warning.png);
+#reminder-notifications > notification[type="warning"] > hbox > .messageImage {
+    width: 32px;
+    height: 32px;
+    fill: #ffbf00;
 }
 
 #reminder-actions-caption,
 #reminder-details-caption,
 #calendar-event-dialog-reminder > .dialog-button-box {
     padding-top: 20px;
 }
 
--- a/mail/components/cloudfile/content/addAccountDialog.xul
+++ b/mail/components/cloudfile/content/addAccountDialog.xul
@@ -43,13 +43,13 @@
 
   <iframe id="accountSettings" class="indent" flex="1" src="chrome://messenger/content/cloudfile/emptySettings.xhtml"/>
   <deck id="messages" class="indent" selectedIndex="-1">
     <hbox id="authorizing">
       <image src="chrome://global/skin/icons/loading.png"/>
       <label>&addAccountDialog.authorizing;</label>
     </hbox>
     <hbox id="error">
-      <image src="chrome://global/skin/icons/warning-16.png"/>
+      <image src="chrome://global/skin/icons/warning.svg"/>
       <label>&addAccountDialog.error;</label>
     </hbox>
   </deck>
 </dialog>
--- a/mail/components/im/themes/imAccounts.css
+++ b/mail/components/im/themes/imAccounts.css
@@ -153,17 +153,17 @@ richlistitem[state="connected"] .account
 }
 richlistitem[state="disconnected"] .accountStateIcon {
   list-style-image: url("chrome://chat/skin/offline-16.png");
 }
 richlistitem[state="connecting"] .accountStateIcon {
   list-style-image: url("chrome://global/skin/icons/loading.png");
 }
 richlistitem[error="true"] .accountStateIcon {
-  list-style-image: url("chrome://global/skin/icons/warning-16.png");
+  list-style-image: url("chrome://global/skin/icons/warning.svg");
 }
 
 
 #statusArea {
   margin: 0 0;
   padding: 0 0;
 }
 
--- a/mail/themes/linux/mail/accountCreation.css
+++ b/mail/themes/linux/mail/accountCreation.css
@@ -36,17 +36,17 @@ menulist[disabled="true"] > .menulist-dr
 
 #outgoing_protocol {
   padding: 6px;
 }
 
 .warningicon {
   width: 16px;
   height: 16px;
-  background: url("chrome://global/skin/icons/warning-16.png") no-repeat;
+  background: url("chrome://global/skin/icons/warning.svg") no-repeat;
 }
 
 .insecureLarry {
   height: 64px;
   width: 64px;
   padding: 0;
   background: url("chrome://global/skin/icons/sslWarning.png") no-repeat;
 }
@@ -210,17 +210,18 @@ menulist[editable="true"] > .menulist-dr
   font-weight: bold;
 }
 
 #status_area[status=error] #status_msg {
   font-weight: bold;
 }
 
 #status_area[status=error] #status_img_before {
-  background: url("moz-icon://stock/gtk-dialog-warning?size=menu") no-repeat;
+  background: url("chrome://global/skin/icons/warning.svg") no-repeat;
+  background-position-y: 1px;
   width: 16px;
   height: 16px;
 }
 
 #status_area[status=loading] #status_img_after {
   background: url("chrome://global/skin/icons/loading.png") no-repeat;
   width: 16px;
   height: 16px;
--- a/mail/themes/osx/mail/accountCreation.css
+++ b/mail/themes/osx/mail/accountCreation.css
@@ -26,17 +26,17 @@
 
 /* Missing:
  * #config_status_title
  */
 
 .warningicon {
   width: 16px;
   height: 16px;
-  background: url("chrome://global/skin/icons/warning-16.png") no-repeat;
+  background: url("chrome://global/skin/icons/warning.svg") no-repeat;
 }
 
 .insecureLarry {
   height: 64px;
   width: 64px;
   padding: 0;
   background: url("chrome://global/skin/icons/sslWarning.png") no-repeat;
 }
@@ -189,17 +189,18 @@ menulist {
   font-weight: bold;
 }
 
 #status_area[status=error] #status_msg {
   font-weight: bold;
 }
 
 #status_area[status=error] #status_img_before {
-  background: url("chrome://global/skin/icons/warning-16.png") no-repeat;
+  background: url("chrome://global/skin/icons/warning.svg") no-repeat;
+  background-position-y: 1px;
   width: 16px;
   height: 16px;
   margin-right: 1em;
 }
 
 #status_area[status=loading] #status_img_after {
   background: url("chrome://global/skin/icons/loading.png") no-repeat;
   width: 16px;
--- a/mail/themes/windows/jar.mn
+++ b/mail/themes/windows/jar.mn
@@ -67,16 +67,17 @@ classic.jar:
   skin/classic/messenger/activity/defaultProcessIcon.png      (mail/activity/defaultProcessIcon.png)
   skin/classic/messenger/activity/deleteMailIcon.png          (mail/activity/deleteMailIcon.png)
   skin/classic/messenger/activity/indexMailIcon.png           (mail/activity/indexMailIcon.png)
   skin/classic/messenger/activity/moveMailIcon.png            (mail/activity/moveMailIcon.png)
   skin/classic/messenger/activity/removeItemIcon.png          (mail/activity/removeItemIcon.png)
   skin/classic/messenger/activity/sendMailIcon.png            (mail/activity/sendMailIcon.png)
   skin/classic/messenger/activity/syncMailIcon.png            (mail/activity/syncMailIcon.png)
   skin/classic/messenger/activity/undoIcon.png                (mail/activity/undoIcon.png)
+  skin/classic/messenger/activity/warning.png                 (mail/activity/warning.png)
   skin/classic/messenger/addressbook/addressbook.css          (mail/addrbook/addressbook.css)
   skin/classic/messenger/addressbook/abContactsPanel.css      (mail/addrbook/abContactsPanel.css)
   skin/classic/messenger/addressbook/cardDialog.css           (mail/addrbook/cardDialog.css)
   skin/classic/messenger/addressbook/abResultsPane.css        (mail/addrbook/abResultsPane.css)
   skin/classic/messenger/addressbook/icons/abcard.png         (mail/addrbook/abcard.png)
   skin/classic/messenger/addressbook/icons/addrbook.png       (mail/addrbook/addrbook.png)
   skin/classic/messenger/addressbook/icons/ablist.png         (mail/addrbook/ablist.png)
   skin/classic/messenger/addressbook/icons/contact-generic.png             (mail/addrbook/contact-generic.png)
--- a/mail/themes/windows/mail/accountCreation.css
+++ b/mail/themes/windows/mail/accountCreation.css
@@ -51,17 +51,17 @@ menulist[disabled="true"] > .menulist-dr
 
 #config_status_title {
   font-weight: bold;
 }
 
 .warningicon {
   width: 16px;
   height: 16px;
-  background: url("chrome://global/skin/icons/warning-16.png") no-repeat;
+  background: url("chrome://global/skin/icons/warning.svg") no-repeat;
 }
 
 .insecureLarry {
   height: 64px;
   width: 64px;
   padding: 0;
   background: url("chrome://global/skin/icons/sslWarning.png") no-repeat;
 }
@@ -227,17 +227,18 @@ textbox.padded html|*.textbox-input::pla
   font-weight: bold;
 }
 
 #status_area[status=error] #status_msg {
   font-weight: bold;
 }
 
 #status_area[status=error] #status_img_before {
-  background: url("chrome://global/skin/icons/warning-16.png") no-repeat;
+  background: url("chrome://global/skin/icons/warning.svg") no-repeat;
+  background-position-y: 1px;
   width: 16px;
   height: 16px;
 }
 
 #status_area[status=loading] #status_img_after {
   background: url("chrome://global/skin/icons/loading.png") no-repeat;
   width: 16px;
   height: 16px;
--- a/mail/themes/windows/mail/activity/activity.css
+++ b/mail/themes/windows/mail/activity/activity.css
@@ -197,17 +197,17 @@ activity-warning .statusText {
   list-style-image: url(chrome://global/skin/icons/Error.png);
 }
 
 .processIconBox image {
   list-style-image: url(chrome://messenger/skin/activity/defaultProcessIcon.png);
 }
 
 .warningIconBox image {
-  list-style-image: url(chrome://global/skin/icons/Warning.png);
+  list-style-image: url(chrome://messenger/skin/activity/warning.png);
 }
 
 image.syncMail {
   list-style-image: url(chrome://messenger/skin/activity/syncMailIcon.png);
 }
 
 image.sendMail {
   list-style-image: url(chrome://messenger/skin/activity/sendMailIcon.png);
new file mode 100644
index e69de29bb2d1d6434b8b29ae775ad8c2e48c5391..7daab9c788c78877e1d226d59b62041c0c0096ae
GIT binary patch
literal 1536
zc$@(M2LJhqP)<h;3K|Lk000e1NJLTq001BW001Be1^@s6b9#F8000HYNkl<ZcmeHK
z1C(o77Tr&=N}1|(QnssY+s49JZCm3hv~AnAZTENK&o8^;<Y~^kNxk~^ANE;muN&UI
z&pG?tn{c|W<KG$o83OM4jewusroxB|F1P?|irU&*^!4?XghHXiB9X}AlTJG6nd6T?
zUVG@Fhp?(xRiYw+k*{MMcJRRm-(|PkeX=a$si&U$_R&Wl-4nnV)-iajd-xg~8}IUX
zJif`vE~si8RaI3@d+xdCbpSZ5V~AKTsH|9B5ZNF6e!p*cxC`OIv(Xa?qG`*kP*+!X
z+zBU~aQ#g;-SokxtUzf=Ny%wE&!fNh25i;LW4q>lbS-)lUauFs?6ONIOKL*|<mczF
z5-lw)cNs)IJs+Yp_#OMU*Udps-HmiQZB%Fbi!Qq8*iBV|-76|8b~Un}8oM4vAratp
z*7a=~lUUsID6Cd1w%KNzHUPHu7O-&P!WBSS8$G_$Ao%*L`_WuC3XTJT1IKej%)=!<
zk63g9#l^+J&d$z1kSRB61x_!aNT;I^a1Xpb9-n-!<Nc3iy#I-U#eF)qY80?&!EF!(
z0nN?LCjba+ECIgy`ufX^jlcZjMl?3aFyewg+R%1&KH9I%!wauTfDSa4e}$3Zxu7ZW
z9COSuR|4SHSwJ8VSO)v=zyBW!3k!WCBi&@UCt$Pi>}xigY8a0}k}^EOpNJS8xC@3)
zYHDhZ{q1jmyTgW7pp@b?qt^@OwxhAu23_L_KygrkaSAGmLL!pTjbsViH4dVu`)%f%
zop;`ujpaHCa5$VMnwpyKa=YD_KmToP?*9V13P2<1%zQ@nOoS+8x6uKQo!(2{&)%!7
ztla+m^UptaeJikgX=&N6nx>(zryZey#Ih$a5egB@S=g;<sANzz6@ii(M{C&#mSo3r
z>#etDW4YD>KL7l4@VtPEipo1hQN%|d-hsOEVZb0(!Jw-O2~mtJSquE$Fy!bo<e0H0
zsRD6W;}0Shn?zAjQLwG8?GNi(fzv~wU`Zk|4K02rJWhc~IcSt^IzEn6cm$lTnH8`|
zJZq1x5%kR3T%w@sha1>fQY%l`V~;%o<fXL|;G+}oGUNXK?Bg3Kdzm&TviJm2kx{70
z6m$isE{(GaBoYZoM8e3`EObCKK9lnvMn~qNprF8W)KN#RbSy;z+S)oW6Lg{cAvZUd
z-F809Gmv<JG9DuW4cTA>0?(6x1S}Q{1THffm_gR8g>GQMqMMLka19&FU;p~o*HINd
znB7NTp#r6Wz~-lEnvUN2*P^yqM|zsFPHRTi3|e*#6^htt8!vX)(ha9ooy}e|!JfBn
zwh+BEmh}4pJMOsS3S-F=$gMVp>gt-ioK6?b@dpTbKSyeMg8Hs!h*gG+5L)-yK7pGq
ziQ~Ep!YB)*07kyN2o4S&P={_|UZK`*f9|>G9=j?J>`p&M*w}>!ZiL?+$z;n&O_<Lg
zd)bBqPH^M6b39nkBW1)b1I}zoL%}qbbY;eJ%PqGw$I=vFIJLOA<W6Iq^WCGU%3Ww?
zPFO+2@R)>a?h`N_R`8F3G_HHdMz8DH>`ii0j0C_k_9)^smXu`huYdjPA80C;slYGj
zye~;6!?4Bfg;UIKwp`f>rVk`ZVg@w83lbxp1!j>IQtwyM{rxR0S-RZVZD|U~%gg_{
z;mEEz_fz&^lV~~PnO-Px%Q#|p6)u+x1$hn}xl1ZTtYD?&&@-rVe1PexK6YD@i2bmX
zBRQk@7Ml;<;c_J634ntEwyZF{I<U<aKx?D%mH<)k;_E&ybVg)i=D8X9!2nf@NA^JS
zn*zGKyIv?P+-&Di*%4IQW6knrR%xXxZNiRWRaIA5*YsL%Z*T0&FTZ@({AWa)N#~t+
zVFw)+htM@zW{}f8U=R_+ETm;#qh7Mj)&cC~B=XOmdFGj`UVr`d$BA2>5isKi55dhU
mvuQ#hL<nJmW^n)b@A#kBbwUdbWmI4Q0000<MNUMnLSTYYOzqYH
--- a/mailnews/base/content/newsError.xhtml
+++ b/mailnews/base/content/newsError.xhtml
@@ -17,17 +17,17 @@
 ]>
 
 <html xmlns="http://www.w3.org/1999/xhtml">
   <head>
     <title>&newsError.title;</title>
     <link rel="stylesheet" href="chrome://global/skin/netError.css" type="text/css" media="all" />
     <!-- If the location of the favicon is changed here, the FAVICON_ERRORPAGE_URL symbol in
          toolkit/components/places/src/nsFaviconService.h should be updated. -->
-    <link rel="icon" type="image/png" id="favicon" href="chrome://global/skin/icons/warning-16.png"/>
+    <link rel="icon" type="image/png" id="favicon" href="chrome://global/skin/icons/warning.svg"/>
     <script type="application/javascript"><![CDATA[
       // Error url must be formatted like this:
       //   about:newserror?r=response&m=messageid&k=messagekey&f=folderuri
       // "r" is required; "m" and "f" are optional, but "k" always comes with "m".
 
       var folderUri;
 
       function removeExpired() {