Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

VoiceOver reads Tab elements as "Tab Description" #30689

Closed

Conversation

adkenyon
Copy link
Contributor

@adkenyon adkenyon commented Jan 5, 2021

Summary

On iOS, VoiceOver reads Tab accessibility role elements as "tab description" rather than "tab". See issue #30589 and #30610

Looking at the file history the bug appears to have been introduced in #27995

Changelog

[iOS] [Fixed] - Tab Accessibility Role had incorrect localization string

Test Plan

Tested with voice over using the RNTester app

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Jan 5, 2021
@react-native-bot react-native-bot added Bug Platform: iOS iOS applications. labels Jan 5, 2021
@analysis-bot
Copy link

Platform Engine Arch Size (bytes) Diff
ios - universal n/a --

Base commit: 0db56f1

@analysis-bot
Copy link

Platform Engine Arch Size (bytes) Diff
android hermes arm64-v8a 8,898,886 +14,173
android hermes armeabi-v7a 8,392,350 +9,243
android hermes x86 9,388,134 +15,047
android hermes x86_64 9,333,802 +15,678
android jsc arm64-v8a 10,354,231 +14,170
android jsc armeabi-v7a 9,830,802 +9,249
android jsc x86 10,404,978 +15,043
android jsc x86_64 10,990,921 +15,695

Base commit: 0db56f1

@dulmandakh dulmandakh requested a review from fkgozali January 6, 2021 01:47
Copy link
Contributor

@dulmandakh dulmandakh left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

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

@mdvacca has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@mdvacca
Copy link
Contributor

mdvacca commented Jan 15, 2021

lgtm, importing to review and land

@facebook-github-bot
Copy link
Contributor

@mdvacca merged this pull request in 80a1095.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. Merged This PR has been merged. Platform: iOS iOS applications.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants