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-24225] Fire close event on Ti.UI.Window close #8685

Merged
merged 2 commits into from
Dec 22, 2016

Conversation

garymathews
Copy link
Contributor

@garymathews garymathews commented Dec 14, 2016

  • close event is not fired when a Titanium.UI.Window is closed
  • This should fix the failing close event is fired mocha test
  • Add argument validation to BufferProxy.append()
TEST CASE
var b = Ti.UI.createWindow({backgroundColor: 'red'}),
    w = Ti.UI.createWindow({backgroundColor: 'yellow'});
 
w.addEventListener('close', function () {
    b.setBackgroundColor('green');
});
b.addEventListener('open', function () {
    w.addEventListener('open', function() {
        setTimeout(function () {
            w.close();
        }, 500);
    });
    w.open();
});
 
b.open();
  • Window should be green

JIRA Ticket

@garymathews
Copy link
Contributor Author

test this please

@garymathews
Copy link
Contributor Author

test this please

@sgtcoolguy
Copy link
Contributor

sgtcoolguy commented Dec 14, 2016

Just a note that the Jenkins build failed a handful of network related tests due to firewall issues on the internal build agent. I'm working on getting the sites used in our unit tests whitelisted.

@garymathews
Copy link
Contributor Author

Ah okay, thanks!

@garymathews
Copy link
Contributor Author

test this please

@garymathews garymathews force-pushed the TIMOB-24225 branch 2 times, most recently from 7c53179 to 1477ff8 Compare December 21, 2016 17:15
@garymathews
Copy link
Contributor Author

All tests are passing, merging.

@garymathews garymathews merged commit b5feb9a into tidev:master Dec 22, 2016
@frankieandone
Copy link
Contributor

Didn't you basically undo the changes from PR for JIRA ticket?

@garymathews
Copy link
Contributor Author

@fmerzadyan no, look at the changes.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants