I am trying this code:
function view_mail_popup_close()
{
setTimeout("function () { $('#popupbox').fadeOut('slow'); }",200);
setTimeout("function () { window.location='view_mail.php' }",800);
}
I want to execute it onclick of a link, but only fading function works!
Can someone tell me why my redirect function not working?
I'm kind of surprised that either of them works, because you're giving setTimeout a string that defines a function without calling it; if you give setTimeout a string, it essentially does an eval on the string when the timeout occurs, which in theory would create but not call the function. (Edit: And I've confirmed that: http://jsbin.com/uvuje5)
It's almost never correct or necessary to give setTimeout a string; instead, give it a function:
function view_mail_popup_close()
{
setTimeout(function () { $('#popupbox').fadeOut('slow'); },200);
setTimeout(function () { window.location='view_mail.php'; },800);
}
There, the function is created immediately and the reference to it is given to setTimeout, which will call it when the timeout occurs.
(Off-topic: I've also added a missing semicolon at the end of the window.location = statemenet. JavaScript has semicolon insertion, and so the previous version would work, but I strongly advocate never relying on it.)
Update: As Capsule points out, there's a callback on fadeOut that you probably want to use instead of a second setTimeout:
function view_mail_popup_close()
{
setTimeout(function () {
$('#popupbox').fadeOut('slow', function() {
window.location='view_mail.php';
});
}, 200);
}
fadeOut callback would be the best way to handle this (no need for the second timeout, I'm guessing he's only leaving more time so that the fade has time to complete). Added.
You should not be putting "function(){}" in quotes - if you use quotes then put the JS code directly there. What you are doing is syntactically incorrect:
Uncaught SyntaxError: Unexpected token (
Just kill the quotes and feed function literals. You can nest them as such:
function view_mail_popup_close()
{
setTimeout(function () {
$('#popupbox').fadeOut('slow');
setTimeout(function () { window.location.href='view_mail.php' },600);
},200);
}
If you use quotes, it's slower because it does extra evaluation, and the scope is not kept intact because its defined in global scope, in addition you have to have the DIRECT JS code in there.
If this still doesn't make the page redirect to view_mail.php, please tell us specifically, exactly what happens. If it redirects to a 404/empty page, then you may need to specify root relative, eg href="/view-mail.php" with the leading /.