Skip to content

Bump to [email protected] and disable tabindex#3193

Merged
compulim merged 5 commits intomicrosoft:masterfrom
compulim:bump-ac-1.2.6
Jun 3, 2020
Merged

Bump to [email protected] and disable tabindex#3193
compulim merged 5 commits intomicrosoft:masterfrom
compulim:bump-ac-1.2.6

Conversation

@compulim
Copy link
Copy Markdown
Contributor

@compulim compulim commented Jun 3, 2020

Fixes #1340.

Changelog Entry

Fixed

  • Fixes #1340. Card container should not be focusable if they do not have tapAction, by @compulim in PR #3193

Changed

Description

Card container should not be focusable (no tabIndex="0") if they do not have tapAction. This requires bumping to Adaptive Cards 1.2.6 plus a global settings.

Specific Changes

  • Bump to [email protected]
  • Set AdaptiveCards.GlobalSettings.setTabIndexAtCardRoot to true if the card has tapAction prop, otherwise false
  • Update tests and screenshots that related to "root container is TAB-able"

  • Testing Added

@compulim compulim marked this pull request as ready for review June 3, 2020 13:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[MAS2.4.3]Focus is landing on non-interactive elements after activating the 'Show a simple' Adaptive cards button.

2 participants