-
Notifications
You must be signed in to change notification settings - Fork 24.4k
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
Disable accessibilityState
when TouchableWithoutFeedback
is disabled
#31297
Disable accessibilityState
when TouchableWithoutFeedback
is disabled
#31297
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code analysis results:
eslint
found some issues. Runyarn lint --fix
to automatically fix problems.
Base commit: 73b9f78 |
accessibilityState
when the TouchableWithoutFeedback
is disabled
accessibilityState
when TouchableWithoutFeedback
is disabled
Libraries/Components/Touchable/__tests__/__snapshots__/TouchableWithoutFeedback-test.js.snap
Outdated
Show resolved
Hide resolved
Libraries/Components/Touchable/__tests__/__snapshots__/TouchableWithoutFeedback-test.js.snap
Show resolved
Hide resolved
Fixed the case found on snapshots with your suggestion @kacieb 😊 |
@kacieb has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator. |
Thank you @carloscuesta for this contribution to a more accessible React Native! I would like to give you a shout-out on Twitter and include you in our end-of-month issues update for your contribution. Do you have a Twitter we can tag? |
Sure @amarlette! Here's my twitter handle @crloscuesta |
Summary
Disable
accessibilityState
when theTouchableWithoutFeedback
isdisabled
. This fixes #30953Changelog
[General] [Changed] - Disable
accessibilityState
when theTouchableWithoutFeedback
isdisabled
.Test Plan
Tested the
TouchableWithoutFeedback
component on an Android device