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-16257] add property and remove methods #5244

Conversation

bijupmb
Copy link
Contributor

@bijupmb bijupmb commented Jan 22, 2014

@@ -72,18 +73,6 @@ public void scrollTo(int x, int y) {
}
}

@Kroll.setProperty @Kroll.method
public void setScrollingEnabled(Object enabled)
Copy link
Contributor

Choose a reason for hiding this comment

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

This is incorrect. Why are you deleting these methods? They seem fine. If you want to add this as an accessor, you need to handle the property in processProperties and propertyChanged.

@hieupham007
Copy link
Contributor

This ticket seems irrelevant. I'll close the PR and ask for more information from the ticket owner.

@hieupham007 hieupham007 closed this Mar 6, 2014
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