From 3482361bb14c10ee84e1a5e279b15918ec019173 Mon Sep 17 00:00:00 2001
From: VTEX <gabriel.antiqueira@vtex.com.br>
Date: Fri, 20 Aug 2021 16:45:31 -0300
Subject: [PATCH 1/3] fix unnecessary id

---
 packages/store-ui/src/molecules/Carousel/Carousel.tsx | 7 +------
 1 file changed, 1 insertion(+), 6 deletions(-)

diff --git a/packages/store-ui/src/molecules/Carousel/Carousel.tsx b/packages/store-ui/src/molecules/Carousel/Carousel.tsx
index 58763a3110..f91730dad4 100644
--- a/packages/store-ui/src/molecules/Carousel/Carousel.tsx
+++ b/packages/store-ui/src/molecules/Carousel/Carousel.tsx
@@ -86,12 +86,7 @@ function Carousel({
   const slides = preRenderedSlides.concat(children ?? [], postRenderedSlides)
 
   return (
-    <section
-      id="store-carousel"
-      data-store-carousel
-      data-testid={testId}
-      aria-label="carousel"
-    >
+    <section data-store-carousel data-testid={testId} aria-label="carousel">
       <div
         data-carousel-track-container
         style={{ overflow: 'hidden', width: '100%' }}

From 03073b3a6d0a6fc7d256e5113d253f7f5368ffae Mon Sep 17 00:00:00 2001
From: VTEX <gabriel.antiqueira@vtex.com.br>
Date: Fri, 20 Aug 2021 16:56:51 -0300
Subject: [PATCH 2/3] custom id

---
 .../store-ui/src/molecules/Carousel/Carousel.tsx    | 13 ++++++++++---
 1 file changed, 10 insertions(+), 3 deletions(-)

diff --git a/packages/store-ui/src/molecules/Carousel/Carousel.tsx b/packages/store-ui/src/molecules/Carousel/Carousel.tsx
index f91730dad4..632ac465ba 100644
--- a/packages/store-ui/src/molecules/Carousel/Carousel.tsx
+++ b/packages/store-ui/src/molecules/Carousel/Carousel.tsx
@@ -24,6 +24,7 @@ const createTransformValues = (infinite: boolean, totalItems: number) => {
 }
 
 export interface CarouselProps extends SwipeableProps {
+  id: string
   testId?: string
   infiniteMode?: boolean
   controls?: 'complete' | 'navigationArrows' | 'paginationBullets'
@@ -44,6 +45,7 @@ function Carousel({
     property: 'transform',
   },
   children,
+  id = 'store-carousel',
   ...swipeableConfigOverrides
 }: PropsWithChildren<CarouselProps>) {
   const childrenArray = React.Children.toArray(children)
@@ -86,7 +88,12 @@ function Carousel({
   const slides = preRenderedSlides.concat(children ?? [], postRenderedSlides)
 
   return (
-    <section data-store-carousel data-testid={testId} aria-label="carousel">
+    <section
+      id={id}
+      data-store-carousel
+      data-testid={testId}
+      aria-label="carousel"
+    >
       <div
         data-carousel-track-container
         style={{ overflow: 'hidden', width: '100%' }}
@@ -150,7 +157,7 @@ function Carousel({
         <div data-carousel-controls>
           <Button
             data-left-arrow
-            aria-controls="store-carousel"
+            aria-controls={id}
             aria-label="previous"
             onClick={() => {
               if (sliderState.sliding) {
@@ -164,7 +171,7 @@ function Carousel({
           </Button>
           <Button
             data-right-arrow
-            aria-controls="store-carousel"
+            aria-controls={id}
             aria-label="next"
             onClick={() => {
               if (sliderState.sliding) {

From 5f0542fc5b425cc31e795635cd1f403644b86056 Mon Sep 17 00:00:00 2001
From: VTEX <gabriel.antiqueira@vtex.com.br>
Date: Fri, 20 Aug 2021 16:58:43 -0300
Subject: [PATCH 3/3] optional id prop

---
 packages/store-ui/src/molecules/Carousel/Carousel.tsx | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/packages/store-ui/src/molecules/Carousel/Carousel.tsx b/packages/store-ui/src/molecules/Carousel/Carousel.tsx
index 632ac465ba..fe323f975c 100644
--- a/packages/store-ui/src/molecules/Carousel/Carousel.tsx
+++ b/packages/store-ui/src/molecules/Carousel/Carousel.tsx
@@ -24,7 +24,7 @@ const createTransformValues = (infinite: boolean, totalItems: number) => {
 }
 
 export interface CarouselProps extends SwipeableProps {
-  id: string
+  id?: string
   testId?: string
   infiniteMode?: boolean
   controls?: 'complete' | 'navigationArrows' | 'paginationBullets'