fix: add visible focus state to actions menu for A11y (#841)

vega · Feb 9, 2022 · d71f2b2 · d71f2b2
1 parent 27594b9
commit d71f2b2
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/vega-embed.scss b/vega-embed.scss
@@ -25,7 +25,6 @@
     border-radius: 999px;
     opacity: 0.2;
     transition: opacity 0.4s ease-in;
-    outline: none;
     cursor: pointer;
     line-height: 0px; // For Safari
 
@@ -48,7 +47,7 @@
   }
 
   &:hover summary,
-  &:focus summary {
+  &:focus-within summary {
     opacity: 1 !important;
     transition: opacity 0.2s ease;
   }
@@ -80,7 +79,8 @@
       color: #434a56;
       text-decoration: none;
 
-      &:hover {
+      &:hover,
+      &:focus {
         background-color: #f7f7f9;
         color: black;
       }