fix(a11y): set aria-haspopup when bal-popup button

baloise · Jan 10, 2025 · f4b16b0 · f4b16b0
1 parent 567c987
commit f4b16b0
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 0 deletions.
diff --git a/.changeset/wise-swans-chew.md b/.changeset/wise-swans-chew.md
@@ -0,0 +1,5 @@
+---
+'@baloise/ds-core': patch
+---
+
+**a11y**: set aria-haspopup for popup buttons
diff --git a/packages/core/src/components/bal-button/bal-button.tsx b/packages/core/src/components/bal-button/bal-button.tsx
@@ -100,6 +100,11 @@ export class Button implements ComponentInterface {
    */
   @Prop() rounded = false
 
+  /**
+   * If `true` the button is a popup.
+   */
+  @Prop() balPopup = undefined
+
   /**
    * Name of the left button icon
    */
@@ -174,6 +179,14 @@ export class Button implements ComponentInterface {
     ])
   }
 
+  componentDidLoad() {
+    if (this.el.getAttribute('bal-popup') && !this.aria?.haspopup) {
+      this.aria = {
+        haspopup:"true"
+      }
+    }
+  }
+
   componentDidRender() {
     this.balDidRender.emit()
   }