Skip to content

Commit

Permalink
fix(overflow-menu): update semantic attributes in OverflowMenuItem
Browse files Browse the repository at this point in the history
  • Loading branch information
metonym committed Jul 14, 2021
1 parent ac4dff0 commit af1caf9
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions src/OverflowMenu/OverflowMenuItem.svelte
Original file line number Diff line number Diff line change
Expand Up @@ -43,16 +43,17 @@
$: primaryFocus = $focusedId === id;
$: buttonProps = {
role: "menuitem",
tabindex: "-1",
title: requireTitle ? text : undefined,
class: "bx--overflow-menu-options__btn",
disabled: href ? undefined : disabled,
href: href ? href : undefined,
title: requireTitle ? text : undefined,
};
</script>

<li
role="menuitem"
role="none"
id="{id}"
class:bx--overflow-menu-options__option="{true}"
class:bx--overflow-menu--divider="{hasDivider}"
Expand Down

0 comments on commit af1caf9

Please sign in to comment.