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-11298-propagate layout properties in animation back to proxy after... #3845

Merged
merged 1 commit into from Feb 12, 2013

Conversation

salachi
Copy link
Contributor

@salachi salachi commented Feb 6, 2013

TIMOB-11298 - Propagate the layout properties from animation after the animation is completed.

|| TiC.PROPERTY_BACKGROUND_COLOR.equals(key)) {
viewProxy.setProperty((String) key, options.get(key));
}
}
Copy link
Contributor

Choose a reason for hiding this comment

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

Is there a reason why we are only propagating the layout properties? Can't we just call applyProperties() on the view proxy?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I had looked at it but the applyProperties takes everything from the options and call setProperty. Some of these properties are very specific to animation and doesn't make sense to the View.

@ayeung
Copy link
Contributor

ayeung commented Feb 12, 2013

Code reviewed and functionally tested. Request Accepted

ayeung pushed a commit that referenced this pull request Feb 12, 2013
TIMOB-11298-propagate layout properties in animation back to proxy after...
@ayeung ayeung merged commit 9f3e449 into tidev:master Feb 12, 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