-
-
Notifications
You must be signed in to change notification settings - Fork 1.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
[TIMOB-13243] iOS: center wrapperview when required #4177
Conversation
…ile into timob-13243
Code Reviewed and APPROVED |
Code Reviewed and approved Functionally Tested and approved |
contentSize.size.width = MAX(contentSize.size.width,viewBounds.size.width); | ||
contentSize.size.height = MAX(contentSize.size.height,viewBounds.size.height); | ||
//contentSize.size.width = MAX(contentSize.size.width,viewBounds.size.width); | ||
//contentSize.size.height = MAX(contentSize.size.height,viewBounds.size.height); |
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.
If this is gone, delete it. Git history will know of its passing.
CR approved, minor quibble, but other than that, looks good. |
On Second look of KS .. we have a regression on ScrollView > ScrollView TextArea. |
Closing this PR. Causing too many regressions |
Fixes TIMOB-13243
Regression Path
http://jira.appcelerator.org/browse/TIMOB-10330
http://jira.appcelerator.org/browse/TIMOB-7958
http://jira.appcelerator.org/browse/TIMOB-11610
http://jira.appcelerator.org/browse/TIMOB-11019
http://jira.appcelerator.org/browse/TIMOB-11707
http://jira.appcelerator.org/browse/TIMOB-11476
http://jira.appcelerator.org/browse/TIMOB-11477
http://jira.appcelerator.org/browse/TIMOB-11479
http://jira.appcelerator.org/browse/TIMOB-11377
KS->Base UI->Views->ScrollView