Code structure for a login modal dialog. I'm using twitter mobile and I have a login-dialog.  login-Dialog is a modal dialog.  To call it the user calls loginDialog.show();

I'm not all that comfortable with the coding style I've adopted (i'm very much self schooled in JavaScript) and I'd be interested to hear peoples opinions on the code structure and process handling.  For example I always seem to use expressive functions instead of function declarations.  (some of the nitty gritty is ommitted for focus).  

var loginDialog = (function($)
{
    /**
     * Essentially the init function
     */ 
    var show = function()
    {       
        // This actually shows the dialog
        $("#login-dialog").modal({backdrop: "static", keyboard: false});
        applyBindings();
    }

    var hide = function()
    {
        $("#login-dialog").modal("hide");
    }

    var applyBindings = function()
    {
        $("#login-dialog .btn-primary").click(onLoginClick);
    };

    var onLoginClick = function()
    {
       // validate form, attempt login...
       // logginIn is a jQuery Deferred.
    var loggingIn = myApi.login(username, password);
    loggingIn.done( handleResponse );
    loggingIn.fail( handleError );
    };

    var validate = function(username, password)
    {
       // simple form validation
    };


    var handleResponse = function(json)
    {
            // check json psuedo code
    if (json != "Ok") 
    {
        handleError();
    }
    else
    {

        handleSuccess();
    }
    };


    var handleError = function(msg)
    {       
        msg = msg || "There was a problem communicating with the server";

        var tmpl = _.template($("#alert-tmpl").html(), { head: "Error", body: msg});

        $("#login-dialog form").append(tmpl);
        $("#login-dialog .btn-primary").removeAttr("disabled");
    };

    var handleSuccess = function() 
    {       
        hide();
    };


    return {
        show: show,
        hide: hide
    }

})(jQuery);
First off, I'd retrieve all the elements right away, instead of running $(...) again and again:

var loginDialog = (function($) {
  var dialog, form, button, template;

  // Get the elements once, on page load
  $(function () {
    dialog = $("#login-dialog");
    form   = dialog.find("form");
    button = dialog.find(".btn-primary");
    template = $("#alert-tmpl");
  });

  function show() {
    dialog.modal({
      backdrop: "static",
      keyboard: false
    });
  }

  // ... et cetera ...

})(jQuery);


As you can see, I've also put the variable declaration first. If you use expressive functions, you should do the same with all those variables too (which is why I usually prefer to use standard function declarations instead - fewer variables to declare).

(oh, and, yes, I've moved the braces around, but we've covered that in the comments :)

Generally, though, it seems quite fine, and well-structured. There are some functions that could be dropped, since they just call a different function - for instance handleSuccess which is basically an alias for hide. On the other hand it's nicely descriptive to have a handleSuccess function. And if you want to more on success than simply call hide, it's of course nice to have encapsulated.

Lastly, in handleError you .append the error message to the form, so if there are more errors the form will keep growing. Don't know if that's by design, but I imagine you might want to only show one error at a time (and perhaps even remove it on success).