Merge pull request #38 from NASA-IMPACT/carousel-fix

Double button on carousel fix
NASA-IMPACT · Nov 17, 2023 · 7fa5afe · 7fa5afe
2 parents b8818fd + d798343
commit 7fa5afe
Showing 1 changed file with 4 additions and 16 deletions.
diff --git a/overrides/common/embedded-video-carousel.tsx b/overrides/common/embedded-video-carousel.tsx
@@ -136,8 +136,10 @@ export default function Carousel({ items }: EmbeddedVideosPropType) {
             <DotGroup
               renderDots={(props) => {
                 return (
-                  <span>
-                    {props.currentSlide + 1}/{items.length + 1}
+                  <span aria-live="polite">
+                    <SROnly>Current slide: </SROnly>
+                    {props.currentSlide + 1}
+                    <SROnly>Total of </SROnly>/{items.length + 1}
                   </span>
                 );
               }}
@@ -178,20 +180,6 @@ export default function Carousel({ items }: EmbeddedVideosPropType) {
             </Slider>
           </FeaturedList>
         </div>
-        {items.length > 1 && (
-          <ButtonGroup
-            topHeight={height}
-            role="group"
-            aria-label="Slide controls"
-          >
-            <ButtonBackStyled>
-              <CollecticonChevronLeft title="Go to previous slide" meaningful />
-            </ButtonBackStyled>
-            <ButtonNextStyled>
-              <CollecticonChevronRight title="Go to next slide" meaningful />
-            </ButtonNextStyled>
-          </ButtonGroup>
-        )}
       </CarouselProvider>
     </Lazyload>
   );