[Android] Checkbox to inherit from AppCompatCheckBox#18318
Conversation
Generated by 🚫 dangerJS |
|
@vonovak Thanks for the PR! React Native supports BUCK on Android, this is the part of the build that fails. You will need to add the dependency to the BUCK file in the checkbox folder. You can probably have a look at another component that uses the support library to figure out what you need to add. |
2ad658f to
3d68948
Compare
3d68948 to
55b7d71
Compare
|
@janicduplessis thanks for the quick response, since I'm not experienced in using buck myself, I somehow ignored the buck files. the PR should be good for merge. |
|
Looks good! I’ll check out your other PR too soon. @facebook-github-bot shipit |
|
Something went wrong executing that command, @hramos could you take a look? |
facebook-github-bot
left a comment
There was a problem hiding this comment.
@janicduplessis is landing this pull request. If you are a Facebook employee, you can view this diff on Phabricator.
|
I tried to merge this pull request into the Facebook internal repo but some checks failed. To unblock yourself please check the following: Does this pull request pass all open source tests on GitHub? If not please fix those. Does the code still apply cleanly on top of GitHub master? If not can please rebase. In all other cases this means some internal test failed, for example a part of a fb app won't work with this pull request. I've added the Import Failed label to this pull request so it is easy for someone at fb to find the pull request and check what failed. If you don't see anyone comment in a few days feel free to comment mentioning one of the core contributors to the project so they get a notification. |
|
Thanks for the contribution! It looks like I'll need to update some buck related files on our side before I can merge this. I'll add it to my task list. No further action is required on your part -- please do not add commits to this PR now that it has been approved as doing so may prevent it from getting merged. |
|
Looked into this again, the actual error we're seeing is: I need to get back to this again later. |
|
@hramos Any update on this, could be related with the crazyness in https://github.com/facebook/react-native/blob/master/ReactAndroid/src/main/third-party/android/support/v7/appcompat-orig/BUCK. I'm not very familiar with BUCK so I can't really help here. |
facebook-github-bot
left a comment
There was a problem hiding this comment.
@hramos is landing this pull request. If you are a Facebook employee, you can view this diff on Phabricator.
|
The error logs were cleared, so let's try importing again. |
facebook-github-bot
left a comment
There was a problem hiding this comment.
@hramos is landing this pull request. If you are a Facebook employee, you can view this diff on Phabricator.
Motivation
The checkbox components inherits from
android.widget.CheckBox.AppCompatCheckBoxoffers better appearance and support for advanced features (eg. tinting) on older APIs.However, the build fails for some reason; If somebody could shed some light on this, I'd appreciate it.Thanks for the comment, I was being blind and somehow ignored the BUCK file.Test Plan
I have created a simple app with a checkbox.
Screenshot from android 4.1 (current master):
Screenshot after applying the change, also android 4.1:
Related PRs
#18300 (this PR is needed to support tinting on older android api levels)
Release Notes
[ANDROID] [ENHANCEMENT] [Checkbox] - Checkbox inherits from AppCompatCheckBox