-
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
remove ViewHelper, use ViewCompat instead #23280
Conversation
Generated by 🚫 dangerJS |
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.
LGTM
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.
@mdvacca is landing this pull request. If you are a Facebook employee, you can view this diff on Phabricator.
@dulmandakh merged commit c493cfe into |
Summary: ViewHelper has only setBackground method which works identical to ViewCompat.setBackground from Android Support Library. This PR removes ViewHelper, and uses ViewCompat instead. [Android] [Changed] - remove ViewHelper, use ViewCompat instead Pull Request resolved: facebook#23280 Differential Revision: D13950510 Pulled By: mdvacca fbshipit-source-id: 10b5122affac17d4b66fb995339c6715c0871ed0
Summary
ViewHelper has only setBackground method which works identical to ViewCompat.setBackground from Android Support Library. This PR removes ViewHelper, and uses ViewCompat instead.
Changelog
[Android] [Changed] - remove ViewHelper, use ViewCompat instead
Test Plan
CI is green, and everything works as usual.