Skip to content

Commit

Permalink
[JS] Remove role='menubar' and role='menuitem' from `ActionCollec…
Browse files Browse the repository at this point in the history
  • Loading branch information
paulcam206 authored and JeanRoca committed Apr 7, 2022
1 parent 3a6103f commit 505e910
Showing 1 changed file with 0 additions and 7 deletions.
7 changes: 0 additions & 7 deletions source/nodejs/adaptivecards/src/card-elements.ts
Original file line number Diff line number Diff line change
Expand Up @@ -5001,7 +5001,6 @@ class ActionCollection {
let buttonStrip = document.createElement("div");
buttonStrip.className = hostConfig.makeCssClassName("ac-actionSet");
buttonStrip.style.display = "flex";
buttonStrip.setAttribute("role", "menubar");

if (orientation == Enums.Orientation.Horizontal) {
buttonStrip.style.flexDirection = "row";
Expand Down Expand Up @@ -5108,12 +5107,6 @@ class ActionCollection {
action.render();

if (action.renderedElement) {
if (primaryActions.length > 1) {
action.renderedElement.setAttribute("aria-posinset", (i + 1).toString());
action.renderedElement.setAttribute("aria-setsize", primaryActions.length.toString());
action.renderedElement.setAttribute("role", "menuitem");
}

if (hostConfig.actions.actionsOrientation == Enums.Orientation.Horizontal && hostConfig.actions.actionAlignment == Enums.ActionAlignment.Stretch) {
action.renderedElement.style.flex = "0 1 100%";
}
Expand Down

0 comments on commit 505e910

Please sign in to comment.