Returning false on link click jquery

I wish to have a link which opens popup when clicked however I wish it to open a page i开发者_Python百科n a new window if the user doesn't have JS enabled.

The following doesn't seem to work,

<a id="tac-link" target="_blank" href="tac.html">terms and conditions</a>

function popUp(URL) {
day = new Date();
id = day.getTime();
    eval("page" + id + " = window.open(URL, '" + id + "', 'toolbar=0,scrollbars=1,location=0,statusbar=0,menubar=0,resizable=0,width=550,height=600,left = 445,top = 150');");
    }

$('tac-link').click(function() {
    popUp('tac.html');
    return false;
});

---

I think you just have the id selector wrong. Try changing $('tac-link') to $('#tac-link').

Another approach:

$('a[target="_blank"]').click(function(e) {
    window.open( this.href );
    e.preventDefault();
});

---

In order to make this functionality more browser-compatible, you need to pass the event object to the click handler, then call e.preventDefault();.

example:

$( '#button' ).click( function( e ) {
    //... your code...

    e.preventDefault();
    return false;
} );

---

Also note that event.preventDefault is not present in all browsers and will cause an error in things like IE7 (if memory serves.) To avoid that, use a general if-statement:

if (e.preventDefault) {  
    e.preventDefault();  
}  
e.returnValue = false;

This will prevent the error from being thrown. I believe jQuery uses this approach as well.

---

You simply missed the # indicating it's an id and not a node name.

$('#tac-link')

PS - I wouldn't recommend using eval, why not just store all the page variables in an object?

var pages = {};

function popUp( url ) {
    var id = +new Date;
    pages[id] = window.open( url );
}