Is there something wrong with my code?

Question

I don't think there is but this wont load inside facebox:

 $("#email").submit(function(event) {
    event.preventDefault();
    if ($("#tinput").val() == "") {
        $('#error').append('You need to select a template.<br />');
    } else if ($("#tsubject").val() == ""){
        $('#error').append('Subject cannot be blank.<br />');
    } else if ($("#tmessage").val() == "") {
        $('#error').append('Message cannot be blank.<br />');
    } else {
        $('#ecanvas').html('<center><img src="/templates/v2/images/loading.gif" /></center>');
        var formdata = $(this).serialize();
        $.post("/wizard/email-add", formdata, function (data) {
            $('#ecanvas').html(data);
        });
    return false;
})

This works without any problems:

$("#email").submit(function(event) {
    event.preventDefault();
        $('#ecanvas').html('<center><img src="/templates/v2/images/loading.gif" /></center>');
        var formdata = $(this).serialize();
        $.post("/wizard/email-add", formdata, function (data) {
            $('#ecanvas').html(data);
        });
    return false;
})

I should mention #ecanvas, #error, #tinput, #tsubject and #tmessage all exist.

Answer 1

You failed to close the opening brace of the else