Bug 1401009 - Add border between notification buttons when highlight is disabled. r=johannh, a=sledru
authorMichael Kaply <mozilla@kaply.com>
Tue, 26 Sep 2017 14:25:33 -0500
changeset 431938 2399e64d6af2cb9c8be4aff60f7d8b45c0598df6
parent 431937 3ab379041f96a8a4c135a72b02f8dff9935e7232
child 431939 318aab4f559397c6e661db77f0d8f75ba6bb72e5
push id7843
push userryanvm@gmail.com
push dateThu, 28 Sep 2017 13:51:40 +0000
treeherdermozilla-beta@04e03b0726f9 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersjohannh, sledru
bugs1401009
milestone57.0
Bug 1401009 - Add border between notification buttons when highlight is disabled. r=johannh, a=sledru MozReview-Commit-ID: 4YGdUZstiDk
toolkit/themes/linux/global/notification.css
toolkit/themes/osx/global/notification.css
toolkit/themes/windows/global/notification.css
--- a/toolkit/themes/linux/global/notification.css
+++ b/toolkit/themes/linux/global/notification.css
@@ -6,8 +6,12 @@
 
 %include ../../shared/notification.inc.css
 %include ../../shared/popupnotification.inc.css
 
 .popup-notification-button:-moz-focusring {
   outline: 1px -moz-dialogtext dotted;
   outline-offset: -5px;
 }
+
+.popup-notification-button[anonid="secondarybutton"]:not([hidden="true"])  ~ .popup-notification-button[default]:not([highlight="true"]) {
+  border-inline-start: 1px solid var(--panel-separator-color);
+}
--- a/toolkit/themes/osx/global/notification.css
+++ b/toolkit/themes/osx/global/notification.css
@@ -11,8 +11,12 @@
 .popup-notification-button:-moz-focusring {
   outline: 2px -moz-mac-focusring solid;
   outline-offset: -2px;
 }
 
 .popup-notification-warning {
   color: #aa1b08;
 }
+
+.popup-notification-button[anonid="secondarybutton"]:not([hidden="true"])  ~ .popup-notification-button[default]:not([highlight="true"]) {
+  border-inline-start: 1px solid var(--panel-separator-color);
+}
--- a/toolkit/themes/windows/global/notification.css
+++ b/toolkit/themes/windows/global/notification.css
@@ -19,8 +19,12 @@
   }
 }
 
 /* Swap the default and secondary action, because Windows
  * platform conventions put the default action on the left. */
 .popup-notification-button[default] {
   order: -1;
 }
+
+.popup-notification-button[anonid="secondarybutton"]:not([hidden="true"])  ~ .popup-notification-button[default]:not([highlight="true"]) {
+  border-inline-end: 1px solid var(--panel-separator-color);
+}