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-15314-set the layout params to borderview if view has border #4778

Merged
merged 1 commit into from Oct 13, 2013

Conversation

salachi
Copy link
Contributor

@salachi salachi commented Oct 10, 2013

If the parent is border view, set the layout params to the border view.

https://jira.appcelerator.org/browse/TIMOB-15314

@hieupham007
Copy link
Contributor

I tested this on 2.2 and 4.2.x devices. On 2.2, it works fine (I can repro the crash). On 4.2.x, I dont see the box anymore after it animates ( I do see the box on 2.2 after it animates).

@hieupham007
Copy link
Contributor

Are you seeing the same behavior?

@salachi
Copy link
Contributor Author

salachi commented Oct 11, 2013

Since this was pre Honeycomb, I didn't try it with 4.2. I tried it now and you are right, I don't see the box. Since the code change won't affect 4.2, looks like this was already broken for 4.2. Looks like there are some recent changes in this file specific to HoneyComb and later versions.

@hieupham007
Copy link
Contributor

Ok, I opened a new ticket for this issue: https://jira.appcelerator.org/browse/TIMOB-15471. Merging this PR

hieupham007 added a commit that referenced this pull request Oct 13, 2013
TIMOB-15314-set the layout params to borderview if view has border
@hieupham007 hieupham007 merged commit 6f9dc3d into tidev:master Oct 13, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants