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

Fixes to run on Windows Platform #4

Merged
merged 2 commits into from Jan 10, 2017

Conversation

ewanharris
Copy link
Contributor

  • Windows would throw Undefined is not an object (evaluating 'template.properties') on launch due to the constant Ti.UI.LIST_ITEM_TEMPLATE_SUBTITLE not being implemented as it is iOS only.

    • Should this be under Ti.UI.iOS?
    • We also should probably handle that better in Windows
  • Both Windows and Android were crashing due to the Ti.App.iOS.addEventListener call so wrap in if(OS_IOS)

  • Set the backgroundColor for a Window to black on Windows

To verify

  1. Build the app for Android, it should launch without any errors
  2. Build the app for Windows, it should launch without any errors and the text for the different items should be visible.

@hansemannn
Copy link
Contributor

Should this be under Ti.UI.iOS?

I think it's fine for now, maybe Android / Windows come up with an equivalent some time or we provide the template.

Merging for now!

@hansemannn hansemannn merged commit 37980c5 into tidev:master Jan 10, 2017
hansemannn added a commit that referenced this pull request Mar 6, 2018
Fixes to run on Windows Platform
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