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-19919] separates event and callback for onBackPressed #6859

Merged
merged 3 commits into from Jan 5, 2016

Conversation

manumaticx
Copy link
Contributor

Test code in ticket description: AC-96

This allows you to listen to the back button event without influencing the default behavior. Therefor, a new property was added (Ti.UI.Window.onBack) which lets you define a callback that explicitly overrides onBackPressed.

The existing androidback event fires without impact on the behavior. This allows you to execute additional code.

@hieupham007
Copy link
Contributor

Code reviewed and functionally tested. Looks good

hieupham007 added a commit that referenced this pull request Jan 5, 2016
[AC-96] separates event and callback for onBackPressed
@hieupham007 hieupham007 merged commit e4a9e02 into tidev:master Jan 5, 2016
@hansemannn
Copy link
Collaborator

Would be great to have the since tag inside the docs.

@ashcoding
Copy link
Contributor

As mentioned in https://jira.appcelerator.org/browse/TIMOB-19919
Will get this out of 5.4.0 (aka current Master) and get it merged back when we do 6.0.0. Will be reverting it right now.

@ashcoding
Copy link
Contributor

Revert button not able to create PR automatically, will be required to manually create a PR to revert the changes.

@ashcoding ashcoding changed the title [AC-96] separates event and callback for onBackPressed [TIMOB-19919] separates event and callback for onBackPressed Feb 26, 2016
@ashcoding
Copy link
Contributor

PR: #7780

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