refactor(esl-carousel): fix a11y for carousel control

exadel-inc · Jan 29, 2024 · afaaa40 · afaaa40
1 parent a5a3b78
commit afaaa40
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/src/modules/esl-carousel/core/esl-carousel.slide.ts b/src/modules/esl-carousel/core/esl-carousel.slide.ts
@@ -59,7 +59,7 @@ export class ESLCarouselSlide extends ESLBaseElement {
   }
 
   protected updateA11y(): void {
-    this.setAttribute('role', 'group');
+    this.setAttribute('role', 'listitem');
     if (!this.hasAttribute('aria-roledescription')) {
       this.setAttribute('aria-roledescription', 'slide');
     }

diff --git a/src/modules/esl-carousel/core/esl-carousel.ts b/src/modules/esl-carousel/core/esl-carousel.ts
@@ -167,7 +167,9 @@ export class ESLCarousel extends ESLBaseElement {
   }
 
   protected updateA11y(): void {
-    // TODO: update a11y -> check a11y everywhere
+    if (!this.$slidesArea.hasAttribute('role')) {
+      this.$slidesArea.setAttribute('role', 'list');
+    }
   }
 
   @listen({