Merge pull request #381 from bkochendorfer/FXA-8440-tab-focus-close

fix: Make tab focus work
mozilla-iam · Oct 12, 2023 · bdc77d1 · bdc77d1
2 parents b88dfe0 + d2175d8
commit bdc77d1
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/src/html/index.html b/src/html/index.html
@@ -71,11 +71,11 @@
                       class="close-brand-banner"
                       aria-label="Close Banner"
                       type=button
+                      data-handler="mozilla-accounts-pre"
                       >
                       <img
                           src="{{{ cdn }}}/images/x.svg"
                           class="cursor-pointer"
-                          data-handler="mozilla-accounts-pre"
                           alt="Close Banner"
                       />
                   </button>
@@ -110,12 +110,13 @@
                       class="close-brand-banner"
                       aria-label="Close Banner"
                       type=button
+                      data-handler="mozilla-accounts-post"
                       >
                       <img
                           src="{{{ cdn }}}/images/x.svg"
                           class="cursor-pointer"
-                          data-handler="mozilla-accounts-post"
                           alt="Close Banner"
+
                       />
                   </button>
               </div>

diff --git a/src/scss/components/_branding.scss b/src/scss/components/_branding.scss
@@ -78,7 +78,6 @@
     float: right;
     background-color: transparent;
     border-style: none;
-    box-shadow: none;
     cursor: pointer;
   }
 }