Merge pull request #1296 from cdrini/fix/a11y

Make focus outlines show on BR buttons
internetarchive · Jan 5, 2024 · a5eb2c1 · a5eb2c1
2 parents 2dafc4f + 9fdedeb
commit a5eb2c1
Show file tree

Hide file tree

Showing 5 changed files with 9 additions and 13 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -26,7 +26,7 @@
   "private": false,
   "dependencies": {
     "@internetarchive/ia-activity-indicator": "^0.0.4",
-    "@internetarchive/ia-item-navigator": "^2.0.3",
+    "@internetarchive/ia-item-navigator": "^2.1.0",
     "@internetarchive/icon-bookmark": "^1.3.4",
     "@internetarchive/icon-dl": "^1.3.4",
     "@internetarchive/icon-edit-pencil": "^1.3.4",

diff --git a/src/css/_BRmain.scss b/src/css/_BRmain.scss
@@ -20,9 +20,6 @@
     font-weight: 700;
     color: #dedede;
   }
-  a {
-    outline: none;
-  }
   button {
     cursor: pointer;
   }

diff --git a/src/css/_BRnav.scss b/src/css/_BRnav.scss
@@ -103,7 +103,7 @@
     border: none;
     border-radius: 2px;
     background: transparent;
-    outline: none;
+
     @media (hover: hover) {
       /* styles to apply on devices that support hover */
       &:hover .BRtooltip {

diff --git a/src/css/_BRsearch.scss b/src/css/_BRsearch.scss
@@ -164,7 +164,6 @@
       max-width: 160px;
       height: 100%;
       color: white;
-      outline: none;
       border: none;
       padding: 3px 0px 3px 10px;
       margin: 0;