From e0e5b0de96b0bee9d0dacd9cc258f3ff7cbc93e5 Mon Sep 17 00:00:00 2001
From: Rajesh Kumar <glo71317@adobe.com>
Date: Tue, 26 Apr 2022 16:25:42 +0530
Subject: [PATCH 1/6] AC-2493::Items within actions menu are not operable for
 keyboard-only users (Shopping Bag)

---
 .../venia-ui/lib/components/LegacyMiniCart/kebab.module.css   | 4 ++++
 packages/venia-ui/lib/components/LegacyMiniCart/section.js    | 2 +-
 2 files changed, 5 insertions(+), 1 deletion(-)

diff --git a/packages/venia-ui/lib/components/LegacyMiniCart/kebab.module.css b/packages/venia-ui/lib/components/LegacyMiniCart/kebab.module.css
index 8bb845a610..5d91e56e44 100644
--- a/packages/venia-ui/lib/components/LegacyMiniCart/kebab.module.css
+++ b/packages/venia-ui/lib/components/LegacyMiniCart/kebab.module.css
@@ -35,3 +35,7 @@
     composes: border-0 from global;
     composes: outline-none from global;
 }
+
+.kebab:focus {
+    border: 1px solid rgb(115, 115, 115);
+}
diff --git a/packages/venia-ui/lib/components/LegacyMiniCart/section.js b/packages/venia-ui/lib/components/LegacyMiniCart/section.js
index b6b8a2ec1d..fef91ec8be 100644
--- a/packages/venia-ui/lib/components/LegacyMiniCart/section.js
+++ b/packages/venia-ui/lib/components/LegacyMiniCart/section.js
@@ -38,7 +38,7 @@ const Section = props => {
         <li className={classes.menuItem} {...restProps}>
             <button
                 className={classes.button}
-                onMouseDown={onClick}
+                onClick={onClick}
                 data-cy="LegacyMiniCart-Section-button"
             >
                 <Icon classes={iconClasses} size={16} src={iconSrc} />

From 2da62618a34af8dc0538f34e398c81174b40965c Mon Sep 17 00:00:00 2001
From: Rajesh Kumar <glo71317@adobe.com>
Date: Mon, 2 May 2022 15:16:13 +0530
Subject: [PATCH 2/6] AC-2788:updated the snapshot because tests are failing

---
 .../__tests__/__snapshots__/product.spec.js.snap          | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/packages/venia-ui/lib/components/CartPage/ProductListing/__tests__/__snapshots__/product.spec.js.snap b/packages/venia-ui/lib/components/CartPage/ProductListing/__tests__/__snapshots__/product.spec.js.snap
index 12e334ff6e..a961d80102 100644
--- a/packages/venia-ui/lib/components/CartPage/ProductListing/__tests__/__snapshots__/product.spec.js.snap
+++ b/packages/venia-ui/lib/components/CartPage/ProductListing/__tests__/__snapshots__/product.spec.js.snap
@@ -267,7 +267,7 @@ exports[`renders configurable product with options 1`] = `
         >
           <button
             className="button"
-            onMouseDown={[MockFunction]}
+            onClick={[MockFunction]}
           >
             <span
               className="root"
@@ -301,7 +301,7 @@ exports[`renders configurable product with options 1`] = `
         >
           <button
             className="button"
-            onMouseDown={[MockFunction]}
+            onClick={[MockFunction]}
           >
             <span
               className="root"
@@ -600,7 +600,7 @@ exports[`renders out of stock product 1`] = `
         >
           <button
             className="button"
-            onMouseDown={[MockFunction]}
+            onClick={[MockFunction]}
           >
             <span
               className="root"
@@ -899,7 +899,7 @@ exports[`renders simple product correctly 1`] = `
         >
           <button
             className="button"
-            onMouseDown={[MockFunction]}
+            onClick={[MockFunction]}
           >
             <span
               className="root"

From a93a4b0933dd89e7ab6dbf4dd7253436fda2dfd6 Mon Sep 17 00:00:00 2001
From: James Calcaben <calcaben@adobe.com>
Date: Tue, 30 Aug 2022 13:19:43 -0500
Subject: [PATCH 3/6] Fix screen reader not reading menu buttons

---
 packages/venia-ui/lib/components/LegacyMiniCart/kebab.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/packages/venia-ui/lib/components/LegacyMiniCart/kebab.js b/packages/venia-ui/lib/components/LegacyMiniCart/kebab.js
index 30c1b2a19f..18c9ddb0b7 100644
--- a/packages/venia-ui/lib/components/LegacyMiniCart/kebab.js
+++ b/packages/venia-ui/lib/components/LegacyMiniCart/kebab.js
@@ -24,7 +24,7 @@ const Kebab = props => {
             >
                 <Icon src={MoreVerticalIcon} />
             </button>
-            <ul aria-hidden="true" className={toggleClass}>
+            <ul aria-hidden={isOpen ? 'false' : 'true'} className={toggleClass}>
                 {children}
             </ul>
         </div>

From 098b62afbce6dbd67017f8875d599c021811588b Mon Sep 17 00:00:00 2001
From: James Calcaben <calcaben@adobe.com>
Date: Tue, 30 Aug 2022 13:39:45 -0500
Subject: [PATCH 4/6] Enable focus outline on kebab menu

---
 .../lib/components/LegacyMiniCart/kebab.module.css         | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/packages/venia-ui/lib/components/LegacyMiniCart/kebab.module.css b/packages/venia-ui/lib/components/LegacyMiniCart/kebab.module.css
index 24eecbf49b..5e08b0e30f 100644
--- a/packages/venia-ui/lib/components/LegacyMiniCart/kebab.module.css
+++ b/packages/venia-ui/lib/components/LegacyMiniCart/kebab.module.css
@@ -34,9 +34,8 @@
 .kebab {
     composes: bg-white from global;
     composes: border-0 from global;
-    composes: outline-none from global;
-}
 
-.kebab:focus {
-    border: 1px solid rgb(115, 115, 115);
+    composes: focus_outline from global;
+    composes: focus_outline-2 from global;
 }
+

From 334acf0d044f818cbe068865ba23f77f5751e15d Mon Sep 17 00:00:00 2001
From: James Calcaben <calcaben@adobe.com>
Date: Tue, 30 Aug 2022 13:40:30 -0500
Subject: [PATCH 5/6] Enable focus outline on kebab menu

---
 .../venia-ui/lib/components/LegacyMiniCart/kebab.module.css    | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/packages/venia-ui/lib/components/LegacyMiniCart/kebab.module.css b/packages/venia-ui/lib/components/LegacyMiniCart/kebab.module.css
index 5e08b0e30f..c51ab7e95f 100644
--- a/packages/venia-ui/lib/components/LegacyMiniCart/kebab.module.css
+++ b/packages/venia-ui/lib/components/LegacyMiniCart/kebab.module.css
@@ -34,8 +34,5 @@
 .kebab {
     composes: bg-white from global;
     composes: border-0 from global;
-
-    composes: focus_outline from global;
-    composes: focus_outline-2 from global;
 }
 

From 4c8b11b6f3548229b4d849c02567393c9885457c Mon Sep 17 00:00:00 2001
From: James Calcaben <calcaben@adobe.com>
Date: Thu, 1 Sep 2022 09:50:34 -0500
Subject: [PATCH 6/6] Run prettier

---
 packages/venia-ui/lib/components/LegacyMiniCart/kebab.module.css | 1 -
 1 file changed, 1 deletion(-)

diff --git a/packages/venia-ui/lib/components/LegacyMiniCart/kebab.module.css b/packages/venia-ui/lib/components/LegacyMiniCart/kebab.module.css
index c51ab7e95f..62d871b0ef 100644
--- a/packages/venia-ui/lib/components/LegacyMiniCart/kebab.module.css
+++ b/packages/venia-ui/lib/components/LegacyMiniCart/kebab.module.css
@@ -35,4 +35,3 @@
     composes: bg-white from global;
     composes: border-0 from global;
 }
-