Bug 1296128 - Indicate permission panel's 'X' button while focus r?dao draft
authorRicky Chien <ricky060709@gmail.com>
Tue, 23 Aug 2016 21:25:14 +0800
changeset 408354 0d7d8be94a4158f860ded156a3e83b1dd41d7c7d
parent 406450 1a5b53a831e5a6c20de1b081c774feb3ff76756c
child 530098 dce9c66eefcfdb19674eb6f4d6511a41f4e1e60e
push id28204
push userbmo:rchien@mozilla.com
push dateThu, 01 Sep 2016 02:15:25 +0000
reviewersdao
bugs1296128
milestone51.0a1
Bug 1296128 - Indicate permission panel's 'X' button while focus r?dao MozReview-Commit-ID: 8MqXkLpJoKU
browser/themes/osx/controlcenter/panel.css
browser/themes/shared/controlcenter/panel.inc.css
--- a/browser/themes/osx/controlcenter/panel.css
+++ b/browser/themes/osx/controlcenter/panel.css
@@ -12,16 +12,20 @@
 .identity-popup-expander:-moz-focusring {
   padding: 2px;
 }
 
 .identity-popup-expander:-moz-focusring > .button-box {
   @hudButtonFocused@
 }
 
+.identity-popup-permission-remove-button:-moz-focusring {
+  box-shadow: @focusRingShadow@;
+}
+
 #identity-popup-multiView > .panel-viewcontainer > .panel-viewstack > .panel-subviews {
   border-bottom-right-radius: 3.5px;
 }
 
 #identity-popup-multiView > .panel-viewcontainer > .panel-viewstack > .panel-subviews:-moz-locale-dir(rtl) {
   border-bottom-right-radius: 0;
   border-bottom-left-radius: 3.5px;
 }
--- a/browser/themes/shared/controlcenter/panel.inc.css
+++ b/browser/themes/shared/controlcenter/panel.inc.css
@@ -402,37 +402,37 @@ description#identity-popup-content-verif
   border-width: 0;
   border-radius: 50%;
   min-width: 0;
   padding: 2px;
   background-color: transparent;
 }
 
 .identity-popup-permission-remove-button > .button-box {
-  border-width: 0;
   padding: 0;
+  -moz-appearance: none;
 }
 
 .identity-popup-permission-remove-button > .button-box > .button-icon {
   margin: 0;
   width: 16px;
   height: 16px;
   list-style-image: url(chrome://browser/skin/panel-icons.svg#cancel);
   filter: url(chrome://browser/skin/filters.svg#fill);
   fill: graytext;
 }
 
 .identity-popup-permission-remove-button > .button-box > .button-text {
   display: none;
 }
 
 /* swap foreground / background colors on hover */
-.identity-popup-permission-remove-button:hover {
+.identity-popup-permission-remove-button:not(:-moz-focusring):hover {
   background-color: graytext;
 }
 
-.identity-popup-permission-remove-button:hover > .button-box > .button-icon {
+.identity-popup-permission-remove-button:not(:-moz-focusring):hover > .button-box > .button-icon {
   fill: -moz-field;
 }
 
-.identity-popup-permission-remove-button:hover:active {
+.identity-popup-permission-remove-button:not(:-moz-focusring):hover:active {
   background-color: -moz-fieldtext;
 }