-
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
[General] [Removed] - remove accessibilityComponentType and accessibilityTraits props (a11y) #24344
[General] [Removed] - remove accessibilityComponentType and accessibilityTraits props (a11y) #24344
Conversation
3718517
to
1c2bdeb
Compare
Let me see what happens when I import this to FB. It may take some work to migrate internally before we can land this. |
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.
@cpojer has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.
Did all the necessary work at FB, so just waiting until the diff lands (currently some internal build failure unrelated to this). Thanks for sending a PR removing code, those count triple in my book! :) |
@cpojer nice. I'm going to remove those props also in Fabric code. In case the internal failure gets fixed before I remove it, do you mind to hold off the merge? My plan is to do it between today and tomorrow. |
Do you mind doing that in a separate PR? :) |
@cpojer sure. I can do it in a separate PR 👍 |
This pull request was successfully merged by @elucaswork in f70e58f. When will my fix make it into a release? | Upcoming Releases |
#873) facebook/react-native#24344 Remove deprecated `accessibilityComponentType` and `accessibilityTraits` props.
Summary: `accessibilityTraits` and `accessibilityComponentType` were removed on #24344. This PR removes `accessibilityTraits` prop from Fabric as well. [General] [Removed] - Remove `accessibilityTraits` prop from Fabric. Pull Request resolved: #24643 Differential Revision: D15120193 Pulled By: cpojer fbshipit-source-id: 98440ac447569c26205fc38b93ce2591ccf1a3cf
…ccessibilityTraits props (a11y) (facebook#24344)
…s (#873) facebook/react-native#24344 Remove deprecated `accessibilityComponentType` and `accessibilityTraits` props.
Summary
Closes: #24016
React Native 0.57 introduced cross-platform
accessibilityRole
andaccessibilityStates
props in order to replaceaccessibilityComponentType
(for android) andaccessibilityTraits
(for iOS). With #24095accessibilityRole
andaccessibilityStates
will increase, receiving more options, which seems to be a good moment to remove deprecated props.Changelog
Remove deprecated
accessibilityComponentType
andaccessibilityTraits
props.[General] [Removed] - Remove accessibilityComponentType and accessibilityTraits props
Test Plan
Create a view with old props:
Neither VoiceOver nor TalkBack should announce this element as "button".
Documentation update PR: facebook/react-native-website#873
Further work: Remove props from Fabric in a separate PR
cc @marcmulcahy @jessebeach