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-7226] Complete Ti.UI.WebView implementation #1288
Conversation
@@ -37,7 +41,8 @@ define("Ti/UI/View", | |||
l = this.children.length; | |||
for (; i < l; i++) { | |||
if (this.children[i] === view) { | |||
this.children.splice(i, 1); | |||
l = this.children.splice(i, 1); | |||
l[0]._setParent(); |
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.
Did you intend for the parent to be set to undefined?
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.
Yep, when you do a Ti.UI.createView(), the _parent is undefined, so when you remove() a child, it's _parent should be restored to undefined.
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.
Whoops, didn't see the line skip...thought this was part of _insertAt().
Code reviewed and tested. Request accepted. |
[TIMOB-7226] Complete Ti.UI.WebView implementation
No description provided.