-
Notifications
You must be signed in to change notification settings - Fork 24.2k
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
Cleanup accessibility checks #44885
Cleanup accessibility checks #44885
Conversation
This pull request was exported from Phabricator. Differential Revision: D58390430 |
This pull request was exported from Phabricator. Differential Revision: D58390430 |
Summary: Pull Request resolved: facebook#44885 Cleanup accessibility related props checks to ensure we are doing the minimal amount of work. e.g. reduce duplicate `null` checks and shift checks to conditional branches that use them. Changelog: [Internal] Reviewed By: javache Differential Revision: D58390430
78213f3
to
d5aa617
Compare
Base commit: bfb3b70 |
Differential Revision: D58386781
Summary: Pull Request resolved: facebook#44885 Cleanup accessibility related props checks to ensure we are doing the minimal amount of work. e.g. reduce duplicate `null` checks and shift checks to conditional branches that use them. Changelog: [Internal] Reviewed By: javache Differential Revision: D58390430
This pull request was exported from Phabricator. Differential Revision: D58390430 |
d5aa617
to
3a2a4d6
Compare
This pull request has been merged in 54997e4. |
This pull request was successfully merged by @pieterv in 54997e4. When will my fix make it into a release? | How to file a pick request? |
Summary:
Cleanup accessibility related props checks to ensure we are doing the minimal amount of work. e.g. reduce duplicate
null
checks and shift checks to conditional branches that use them.Changelog: [Internal]
Reviewed By: javache
Differential Revision: D58390430