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-19765] Android: Support running on main/UI thread #7332

Merged
merged 1 commit into from Oct 27, 2015

Conversation

sgtcoolguy
Copy link
Contributor

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

This simply looks for a boolean property named "run-on-main-thread" in your tiapp to toggle which thread to use.

⚠️ This differs from ios in that the flag is under the tag in tiapp there. I'm not sure how to access xml values underneath the tiapp tag at runtime here.

Based on that flag we will basically merge the "Main" and "UI" messengers.

@sgtcoolguy sgtcoolguy changed the title Allow toggling running Kroll runtime on the UI or a separate runtime thread via an app property [TIMOB-19765] Android: Support running on main/UI thread Oct 22, 2015
@jhaynie
Copy link
Contributor

jhaynie commented Oct 22, 2015

@cb1kenobi i think will need to add a change to read the XML property directly underneath android in tiapp.xml. I ran into this problem with another hyperloop tiapp.xml change i wanted to make.

@cb1kenobi
Copy link
Contributor

@jhaynie no problem. Create a ticket with what you need and it'll get done.

@sgtcoolguy
Copy link
Contributor Author

@jhaynie @cb1kenobi I opened up https://jira.appcelerator.org/browse/TIMOB-19797 to track the handling of platform-specific XML in tiapp.xml and access it at runtime.

@sgtcoolguy
Copy link
Contributor Author

@hieupham007 Hey Hieu, can you take a look at this and we'll circle back to using the property under the android tag once TIMOB-19797 lands.

@hieupham007
Copy link
Contributor

Code reviewed, looks good.

sgtcoolguy added a commit that referenced this pull request Oct 27, 2015
[TIMOB-19765] Android: Support running on main/UI thread
@sgtcoolguy sgtcoolguy merged commit 0de99da into tidev:master Oct 27, 2015
@sgtcoolguy sgtcoolguy deleted the main-thread branch October 27, 2015 17:43
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

4 participants