Skip to content

Commit

Permalink
fix(ADA-103): Transcript Options button doesn't announce “expanded” o…
Browse files Browse the repository at this point in the history
…r “collapsed (#176)

Issue:
More transcript option button doesn't announce its state “expanded” or “collapsed”.

Fix:
Add aria-expanded on the button.

Solves ADA-103
  • Loading branch information
Tzipi-kaltura authored Feb 6, 2024
1 parent d2bf4f9 commit eb715cd
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions src/components/popover-menu/popover-menu.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -116,6 +116,8 @@ class PopoverMenu extends Component<PopoverMenuProps, PopoverMenuState> {
tabIndex={0}
data-testid="popover-anchor-container"
className={`${styles.popoverAnchorContainer} ${this.state.isOpen ? styles.active : ''}`}
aria-expanded={this.state.isOpen}
aria-controls="popoverContent"
ref={node => {
this._controlElementRef = node;
}}>
Expand All @@ -127,6 +129,7 @@ class PopoverMenu extends Component<PopoverMenuProps, PopoverMenuState> {
className={styles.popoverComponent}
role="menu"
aria-expanded={this.state.isOpen}
id="popoverContent"
ref={node => {
this._popoverElementRef = node;
}}>
Expand Down

0 comments on commit eb715cd

Please sign in to comment.