@jrbl actually, since there are other ways to close the dialog than pushing the close button (escape-key, for example), this isn't the right place to remove disabled, since it won't get called in those circumstances.

We should just call it, as you do for saveXHR.error, in the first line of the success handler.