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.

Sign up for GitHub

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

Jump to bottom

Remove accessibilityStates support from ViewManagers #4593

Closed
NickGerleman opened this issue Apr 14, 2020 · 1 comment · Fixed by #4625
Closed

Remove accessibilityStates support from ViewManagers #4593

NickGerleman opened this issue Apr 14, 2020 · 1 comment · Fixed by #4625
Assignees
@kmelmon
Labels
Area: Accessibility Code Cleanup
Milestone
0.62 (M5)

Comments

@NickGerleman
Copy link
Collaborator

This has gone from removed to deprecated. We should remove support from our ViewManager. Note that we also need to add support for the non-plural accessiblityState which is tracked by a different issue.
@ghost ghost added the Needs: Triage 🔍 New issue that needs to be reviewed by the issue management team (label applied by bot) label Apr 14, 2020
@NickGerleman NickGerleman mentioned this issue Apr 14, 2020
Merged
@chrisglein chrisglein added Area: Accessibility Backport to 0.61 and removed Backport to 0.61 Needs: Triage 🔍 New issue that needs to be reviewed by the issue management team (label applied by bot) labels Apr 16, 2020
@chrisglein
Copy link
Member

Issue for implementing accessibilityState: #4617

@chrisglein chrisglein added this to the 0.62 (M5) milestone Apr 16, 2020
@kmelmon kmelmon mentioned this issue Apr 17, 2020
Merged
@ghost ghost closed this as completed in #4625 Apr 17, 2020
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@kmelmon kmelmon

Labels
Area: Accessibility Code Cleanup
Projects
None yet
Milestone
0.62 (M5)
Development

Successfully merging a pull request may close this issue.

Remove AccessibilityStates kmelmon/react-native-windows
3 participants
@NickGerleman @chrisglein @kmelmon