[6_2_X][TIMOB-25173] Android: Modified layouts to behave more like iOS. #9321
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Changed Android to resolve Size/Fill layout conflicts like iOS and Windows. * Note: Only an issue if parent uses Ti.UI.SIZE and child uses Ti.UI.FILL. * Android used to shrink the parent to fit the min size allowed by the child. * Now increases the size of the child to fill the parent's parent, like iOS/Windows. - Fixed horizontal layouts where views using FILL width would wrongly wrapped to next row. * Only an issue if another view is already in the row. (Resulting FILL width was too big.) - Fixed layouts to prioritize size defined by view's top/left/bottom/right/center properties over FILL and SIZE. * Only applies if at least 2 coordinate properties were provided. * Now matches iOS' behavior. - Constrained child view width to parent's max width like iOS when using wrapping horizontal views. - This commit also resolves the following layout issues: * TIMOB-12996 * TIMOB-15097 * TIMOB-17628 * TIMOB-17984 * TIMOB-19536 * TIMOB-19598 * TIMOB-19814
…t/right/top/bottom/center properties to work like iOS. - Now supports setting view width/height based on position properties relative to parent's remaining space. - Also fixed horizontal wrapping regression introduced in last commit. * Views using Ti.UI.SIZE were wrongly using row's remaing space instead of view's desired width.
mukherjee2
approved these changes
Aug 22, 2017
There was a problem hiding this comment.
Node Version: 6.10.3
NPM Version: 3.10.10
Mac OS: 10.12.4
Appc CLI: 6.2.3
Appc CLI NPM: 4.2.9
Titanium SDK version: 6.2.0 on PR/9321
Appcelerator Studio, build: 4.9.1.201707200100
Xcode 8.3.2
I tested this with the test cases specified in PR 9320 (master version for this ticket). I tested with three devices, Pixel (7.0), Nexus 6 (6.0.1), and GS4 (4.4.4) FR passed for all devices and test cases.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
JIRA: https://jira.appcelerator.org/browse/TIMOB-25173
Cherry Pick of: #9320