Uh oh!
There was an error while loading. Please reload this page.
[General] [Removed] - remove accessibilityComponentType and accessibilityTraits props (a11y) - #24344
[General] [Removed] - remove accessibilityComponentType and accessibilityTraits props (a11y)#24344estevaolucas wants to merge 1 commit into
Conversation
3718517 to
1c2bdebComparecpojer
commented
Apr 8, 2019
Let me see what happens when I import this to FB. It may take some work to migrate internally before we can land this. |
facebook-github-bot
left a comment
There was a problem hiding this comment.
@cpojer has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.
cpojer
commented
Apr 15, 2019
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! :) |
estevaolucas
commented
Apr 15, 2019
@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. |
cpojer
commented
Apr 15, 2019
Do you mind doing that in a separate PR? :) |
estevaolucas
commented
Apr 15, 2019
@cpojer sure. I can do it in a separate PR 👍 |
react-native-bot
commented
Apr 15, 2019
This pull request was successfully merged by @elucaswork in f70e58f. When will my fix make it into a release? | Upcoming Releases |
#873) react/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) (react#24344)
…s (#873) react/react-native#24344 Remove deprecated `accessibilityComponentType` and `accessibilityTraits` props.
…s (#873) react/react-native#24344 Remove deprecated `accessibilityComponentType` and `accessibilityTraits` props.
Summary
Closes: #24016
React Native 0.57 introduced cross-platform
accessibilityRoleandaccessibilityStatesprops in order to replaceaccessibilityComponentType(for android) andaccessibilityTraits(for iOS). With #24095accessibilityRoleandaccessibilityStateswill increase, receiving more options, which seems to be a good moment to remove deprecated props.Changelog
Remove deprecated
accessibilityComponentTypeandaccessibilityTraitsprops.[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: react/react-native-website#873
Further work: Remove props from Fabric in a separate PR
cc @marcmulcahy@jessebeach