Skip to content
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-23391] : iOS View is not getting horizontally wrapped properly #8805

Merged
merged 5 commits into from Feb 15, 2017

Conversation

vijaysingh-axway
Copy link
Contributor

@hansemannn
Copy link
Collaborator

The CR & FT works, but we need to discuss a few more items:

  • Did you test the changes with other horizontal layouts / sizings already?
  • I guess you've chosen this three conditions because of the offsetH that effects the left / right properties? If so, please provide additional examples to catch the other two conditions to be triggered, so we can ensure those also still work.

@ewieberappc We need to test this PR very detailed to avoid layout regressions. It looks goo to me, but you guys might have more test-cases to check with.

@htbryant94 htbryant94 self-requested a review February 9, 2017 14:20
@hansemannn hansemannn added this to the 6.1.0 milestone Feb 11, 2017
Copy link
Contributor

@ewieberappc ewieberappc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

FR Passed. Tested using:
MacOS 10.12 (16A323)
Studio 4.8.1.201701192045
Ti SDK 6.1.0
Appc NPM 4.2.8
Appc CLI 6.1.0
Alloy 1.9.6
Xcode 8.3 (8W120l)

Tested the provided sample code, ran our layout test suites, KitchenSink tests around layout, and any local projects I had that modified layouts in any way. No errors encountered

@ewieberappc ewieberappc merged commit c3c55d3 into tidev:master Feb 15, 2017
@ewieberappc ewieberappc removed the request for review from htbryant94 February 16, 2017 14:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants