Bug 1347651 - Fix .findbar-button OSX focusring. r=mikedeboer draft
authorTim Nguyen <ntim.bugs@gmail.com>
Sat, 25 Mar 2017 13:15:34 +0000
changeset 551296 2033c6a51fed46365ae3524a6f1bd2669391fd6f
parent 551237 3ba88652c62990226e5a8e9f382cd0904a5dbe50
child 621507 7e521f1574346cba6086e42c02444980f87a0d49
push id51013
push userbmo:ntim.bugs@gmail.com
push dateSat, 25 Mar 2017 13:15:59 +0000
reviewersmikedeboer
bugs1347651
milestone55.0a1
Bug 1347651 - Fix .findbar-button OSX focusring. r=mikedeboer MozReview-Commit-ID: JJWxwquwHhN
browser/themes/osx/browser.css
--- a/browser/themes/osx/browser.css
+++ b/browser/themes/osx/browser.css
@@ -628,16 +628,20 @@ toolbar .toolbarbutton-1 > .toolbarbutto
   transition-duration: 250ms;
 }
 
 .findbar-button {
   background: none;
   box-shadow: none;
 }
 
+.findbar-button:-moz-focusring {
+  box-shadow: var(--focus-ring-box-shadow);
+}
+
 toolbar .toolbarbutton-1:not(:-moz-any([type="menu-button"],#back-button,#forward-button)) {
   padding: 0 4px;
 }
 
 .findbar-button:not(:-moz-any([checked="true"],[disabled="true"])):hover,
 toolbar .toolbarbutton-1:not(:-moz-any([type="menu-button"],[disabled],[open],#back-button,#forward-button)):hover,
 toolbar .toolbarbutton-1[type="menu-button"]:not([disabled]) > .toolbarbutton-menubutton-button[open] + .toolbarbutton-menubutton-dropmarker,
 toolbar .toolbarbutton-1[type="menu-button"]:not([disabled]):-moz-any(:hover,[open]) > .toolbarbutton-menubutton-button,
@@ -667,16 +671,20 @@ toolbar .toolbarbutton-1[type="menu-butt
 }
 
 .findbar-button[checked="true"]:not(:active):hover,
 toolbar .toolbarbutton-1[checked]:not(:active):hover {
   background-color: var(--toolbarbutton-checkedhover-backgroundcolor);
   transition: background-color 250ms;
 }
 
+.findbar-button:not([disabled=true])[checked="true"]:-moz-focusring {
+  box-shadow: var(--toolbarbutton-active-boxshadow), var(--focus-ring-box-shadow);
+}
+
 .toolbarbutton-1[type="menu-button"]:not([overflowedItem=true]) {
   padding: 0;
 }
 
 .toolbarbutton-1 > .toolbarbutton-menubutton-button,
 .toolbarbutton-1 > .toolbarbutton-menubutton-dropmarker {
   margin: 0;
 }