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 6586 add support for backgroundRepeat property that will allow tiling of background image #1769

Merged
merged 3 commits into from Mar 21, 2012
Merged

Conversation

rusticphilosopher
Copy link
Contributor

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

Test case attached to ticket. Given the changes made, a pass of the drillbit UI tests and the KS UI tests should be run against both runtimes in addition to the actual test case in ticket.

@hieupham007
Copy link
Contributor

CR + FR in progress

}

} catch (IOException e) {
Log.e(LCAT, "Exception occured when trying to open stream to specified background image: ", e);
Copy link
Contributor

Choose a reason for hiding this comment

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

This log is sort of confusing. Rephrase perhaps?

Copy link
Contributor

Choose a reason for hiding this comment

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

Nvm. Re-read it, and it makes sense now :)

@hieupham007
Copy link
Contributor

Functional test failed. Crash log when launching KS: http://pastie.org/private/pzxzzfbx2f7of5yi3izzg

@hieupham007
Copy link
Contributor

Tested with test case in v8/rhino, ran KS UI tests in v8/rhino, did drillbit on v8/rhino. Request accepted.

marshall added a commit that referenced this pull request Mar 21, 2012
Timob 6586  add support for backgroundRepeat property that will allow tiling of background image
@marshall marshall merged commit cb3d1f8 into tidev:master Mar 21, 2012
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

3 participants