Skip to content
This repository has been archived by the owner on May 24, 2024. It is now read-only.

Terra show hide focusing #3837

Closed
wants to merge 33 commits into from
Closed

Terra show hide focusing #3837

wants to merge 33 commits into from

Conversation

adoroshk
Copy link
Contributor

@adoroshk adoroshk commented Jul 7, 2023

Summary

  • A new focusRef prop was added to pass a ref to the element that should be focused after the full text is shown.
  • When read by Screenreaders, upon expanding the content the screen reader starts reading the content on the newly revealed content when the component is expanded.
  • Documentation was updated to include the new prop to the examples, and an explanation on how to use the new prop was added.
  • Previous functionality was preserved for passivity.

Testing

This change was tested using:

  • WDIO
  • Jest
  • Visual testing (please attach a screenshot or recording)
  • Other (please describe below)
  • No tests are needed

Reviews

In addition to engineering reviews, this PR needs:

  • UX review
  • Accessibility review
  • Functional review

This PR resolves:

UXPLATFORM-9213

@adoroshk adoroshk closed this Jul 7, 2023
@adoroshk adoroshk reopened this Jul 7, 2023
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