Skip to content
This repository was archived by the owner on Jan 5, 2026. It is now read-only.

[Accessibility] [Screen Reader] Enable the screen reader to announce when nodes are expanded and collapsed in the JSON tree view#2299

Merged
tonyanziano merged 3 commits intomicrosoft:mainfrom
southworks:external/fix/southworks/fix-collapsed-expanded-announcements
Oct 27, 2021
Merged

[Accessibility] [Screen Reader] Enable the screen reader to announce when nodes are expanded and collapsed in the JSON tree view#2299
tonyanziano merged 3 commits intomicrosoft:mainfrom
southworks:external/fix/southworks/fix-collapsed-expanded-announcements

Conversation

@luislera
Copy link
Copy Markdown
Contributor

@luislera luislera commented Oct 11, 2021

Fixes ADO Issue: #63920

Describe the issue

If the screen reader is not reading the state as Expanded/Collapsed after selecting the ‘Expand/Collapse’ button present on the ‘JSON’ section under the “Live Chat’ tab, so it would be difficult and confusing for screen reader users to find out the state of the control.

Actual behavior:

The screen reader is not reading the state as Expanded/Collapsed after selecting the ‘Expand/Collapse’ button present on the ‘JSON’ section under the “Live Chat’ tab.

Expected behavior:

The screen reader should be reading the state as Expanded/Collapsed after selecting the ‘Expand/Collapse’ button present on the ‘JSON’ section under the “Live Chat’ tab.

Repro Steps:

  1. Open Bot Framework Emulator installed on the system.
  2. Bot Framework Emulator Application with Welcome screen opens.
  3. Navigate on the welcome screen and select the Open Bot button.
  4. Open A bot Dialog opens, navigate on the dialog and enter inputs in edit fields and select Connect button.
  5. Bot Gets connected and a new Tab opens, navigate to Type a Message field and enter any chat message.
  6. Bot Response as per text entered appears and changes occur in JSON Section.
  7. Navigate on JSON Section.
  8. Verify the issue.

Changes included in the PR

  • Added a DIV element for handling announcements when tree nodes are expanded/collapsed
  • Added a CSS class to position the DIV element offscreen
  • Updated test cases for the Json Viewer component

Screenshots

Test cases executed successfully
imagen

@coveralls
Copy link
Copy Markdown

coveralls commented Oct 11, 2021

Coverage Status

Coverage increased (+0.03%) to 67.841% when pulling 314cde7 on southworks:external/fix/southworks/fix-collapsed-expanded-announcements into 9a794fb on microsoft:main.

Copy link
Copy Markdown
Contributor

@tonyanziano tonyanziano left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good! Just some minor comments.

Comment thread packages/sdk/ui-react/src/widget/collapsibleJsonViewer/collapsibleJsonViewer.scss Outdated
Copy link
Copy Markdown
Contributor

@tonyanziano tonyanziano left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great, thank you!

@tonyanziano tonyanziano merged commit ed0b611 into microsoft:main Oct 27, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants