-
Notifications
You must be signed in to change notification settings - Fork 24k
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
Simplify ReactViewGroup clip to border #44646
Conversation
This pull request was exported from Phabricator. Differential Revision: D57668976 |
Summary: We can remove most of the code for clipping children to border radius, and recalculating paths, in ReactViewGroup, and rely on the padding box path/rect already set. I will move this to something more generic up the stack so other native components can reuse this logic. Changelog: [Internal] Differential Revision: D57668976
a7555a9
to
71c90d8
Compare
This pull request was exported from Phabricator. Differential Revision: D57668976 |
Base commit: 9dfcb9e |
This pull request has been merged in 4294b24. |
This pull request was successfully merged by @NickGerleman in 4294b24. When will my fix make it into a release? | How to file a pick request? |
This pull request has been reverted by 91d12d9. |
Summary: Pull Request resolved: facebook#44646 We can remove most of the code for clipping children to border radius, and recalculating paths, in ReactViewGroup, and rely on the padding box path/rect already set. I will move this to something more generic up the stack so other native components can reuse this logic. Changelog: [Internal] Reviewed By: javache Differential Revision: D57668976 fbshipit-source-id: 8b8cf956dc8689827bccba5e41751b465fd85eeb
Summary:
We can remove most of the code for clipping children to border radius, and recalculating paths, in ReactViewGroup, and rely on the padding box path/rect already set.
I will move this to something more generic up the stack so other native components can reuse this logic.
Changelog: [Internal]
Differential Revision: D57668976