Bug 1530632 - Change popup notification warning color (linux, osx and windows) to the one use in .identity-popup-connection-not-secure r=johannh
authorCarolina Jimenez Gomez <carolina.jimenez.g@gmail.com>
Thu, 28 Feb 2019 17:42:50 +0000
changeset 519823 a16d8a7b8a30178e1c3248f8bd37d95cd9102d4e
parent 519822 f5936ae0c1d6eee5777bc17d539b4db9d77d2ce7
child 519824 0bf89ff871d631476bfe85fc6d26efa1d6b2ec60
push id10862
push userffxbld-merge
push dateMon, 11 Mar 2019 13:01:11 +0000
treeherdermozilla-beta@a2e7f5c935da [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersjohannh
bugs1530632
milestone67.0a1
first release with
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
last release without
nightly linux32
nightly linux64
nightly mac
nightly win32
nightly win64
Bug 1530632 - Change popup notification warning color (linux, osx and windows) to the one use in .identity-popup-connection-not-secure r=johannh r=johannh Differential Revision: https://phabricator.services.mozilla.com/D21434
toolkit/themes/osx/global/notification.css
toolkit/themes/shared/popupnotification.inc.css
toolkit/themes/windows/global/notification.css
--- a/toolkit/themes/osx/global/notification.css
+++ b/toolkit/themes/osx/global/notification.css
@@ -3,15 +3,11 @@
  * file, You can obtain one at http://mozilla.org/MPL/2.0/. */
 
 %include shared.inc
 @namespace url("http://www.mozilla.org/keymaster/gatekeeper/there.is.only.xul");
 
 %include ../../shared/notification.inc.css
 %include ../../shared/popupnotification.inc.css
 
-.popup-notification-warning {
-  color: #aa1b08;
-}
-
 .popup-notification-secondary-button:not([hidden="true"]) ~ .popup-notification-primary-button:not([default]) {
   border-inline-start: 1px solid var(--panel-separator-color);
 }
--- a/toolkit/themes/shared/popupnotification.inc.css
+++ b/toolkit/themes/shared/popupnotification.inc.css
@@ -87,8 +87,12 @@
 
 .popup-notification-dropmarker > .button-box > .button-menu-dropmarker > .dropmarker-icon {
   width: 16px;
   height: 16px;
   list-style-image: url(chrome://global/skin/icons/arrow-dropdown-16.svg);
   -moz-context-properties: fill;
   fill: currentColor;
 }
+
+.popup-notification-warning {
+  color: #d74345;
+}
--- a/toolkit/themes/windows/global/notification.css
+++ b/toolkit/themes/windows/global/notification.css
@@ -8,22 +8,16 @@
 %include ../../shared/popupnotification.inc.css
 
 /* Override default icon size which is too small for this dropdown */
 .popup-notification-dropmarker > .button-box > .button-menu-dropmarker {
   width: 16px;
   height: 16px;
 }
 
-@media (-moz-windows-default-theme) {
-  .popup-notification-warning {
-    color: #aa1b08;
-  }
-}
-
 /* Swap the primary and secondary action, because Windows
  * platform conventions put the primary action on the left. */
 .popup-notification-primary-button {
   order: -1;
 }
 
 .popup-notification-secondary-button:not([hidden="true"]) ~ .popup-notification-primary-button:not([default]) {
   border-inline-end: 1px solid var(--panel-separator-color);