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

[General] [Removed] - remove accessibilityComponentType and accessibilityTraits props (a11y) #24344

Closed

Conversation

estevaolucas
Copy link

@estevaolucas estevaolucas commented Apr 7, 2019

Summary

Closes: #24016

React Native 0.57 introduced cross-platform accessibilityRole and accessibilityStates props in order to replace accessibilityComponentType (for android) and accessibilityTraits (for iOS). With #24095 accessibilityRole and accessibilityStates will increase, receiving more options, which seems to be a good moment to remove deprecated props.

Changelog

Remove deprecated accessibilityComponentType and accessibilityTraits props.

[General] [Removed] - Remove accessibilityComponentType and accessibilityTraits props

Test Plan

Create a view with old props:

<View
  accessible={true} 
  accessibilityComponentType="button"
  accessibilityTraits="button">...
</View>

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

@estevaolucas estevaolucas requested a review from shergin as a code owner April 7, 2019 22:17
@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 Apr 7, 2019
@cpojer
Copy link
Contributor

cpojer 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.

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.

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

@cpojer
Copy link
Contributor

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
Copy link
Author

@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
Copy link
Contributor

cpojer commented Apr 15, 2019

Do you mind doing that in a separate PR? :)

@estevaolucas
Copy link
Author

@cpojer sure. I can do it in a separate PR 👍

@react-native-bot
Copy link
Collaborator

This pull request was successfully merged by @elucaswork in f70e58f.

When will my fix make it into a release? | Upcoming Releases

@react-native-bot react-native-bot added the Merged This PR has been merged. label Apr 15, 2019
@estevaolucas estevaolucas deleted the chore/a11y-remove-old-props branch April 15, 2019 17:59
cpojer pushed a commit to facebook/react-native-website that referenced this pull request Apr 17, 2019
#873)

facebook/react-native#24344

Remove deprecated `accessibilityComponentType` and `accessibilityTraits` props.
facebook-github-bot pushed a commit that referenced this pull request Apr 29, 2019
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
AKB48 pushed a commit to UnPourTous/react-native that referenced this pull request Apr 21, 2020
JackWillie added a commit to JackWillie/react-native-website that referenced this pull request Nov 27, 2022
…s (#873)

facebook/react-native#24344

Remove deprecated `accessibilityComponentType` and `accessibilityTraits` props.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Discussion: removal of accessibilityComponentType and accessibilityTraits properties
4 participants