Bug 1401009 - Add border between notification buttons when highlight is disabled. r=johannh
authorMichael Kaply <mozilla@kaply.com>
Tue, 26 Sep 2017 14:25:33 -0500
changeset 383110 a44a81c459ab2145cb40dfccbbedd85668c247f4
parent 383109 25a686779a940f1478aa2f4d1595f81344caac5c
child 383111 ade76994700a497c337cd70ca360365b58da70c2
push id32583
push userarchaeopteryx@coole-files.de
push dateWed, 27 Sep 2017 09:46:12 +0000
treeherdermozilla-central@5563e7da39b2 [default view] [failures only]
perfherder[talos] [build metrics] [platform microbench] (compared to previous push)
reviewersjohannh
bugs1401009
milestone58.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 1401009 - Add border between notification buttons when highlight is disabled. r=johannh 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);
+}