Skip to content

Commit

Permalink
Don't skip over action buttons when in vertical alignment (microsoft#…
Browse files Browse the repository at this point in the history
…7158)

* only set to not focusable in horizontal alignment

* attempt

* only do horizontal alignment fix

Co-authored-by: Vsevolod <[email protected]>
  • Loading branch information
PankajBhojwani and beervoley authored Apr 6, 2022
1 parent ec0388b commit 646503b
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions source/nodejs/adaptivecards/src/card-elements.ts
Original file line number Diff line number Diff line change
Expand Up @@ -5668,8 +5668,8 @@ class ActionCollection {
let afterSelectedAction = false;

for (const renderedAction of this._renderedActions) {
// Remove actions after selected action from tabOrder, to skip focus directly to expanded card
if (afterSelectedAction) {
// Remove actions after selected action from tabOrder if the actions are oriented horizontally, to skip focus directly to expanded card
if (this._owner.hostConfig.actions.actionsOrientation == Enums.Orientation.Horizontal && afterSelectedAction) {
renderedAction.isFocusable = false;
}

Expand Down

0 comments on commit 646503b

Please sign in to comment.