How can I improve this?
jQuery
$(function() {
$('button').button();
$('#tabs').tabs();
$('.ui-state-error').hide();
$('.btnlogin').click(function() {
$('#login .ui-state-error').hide();
if($('#login_email').val().trim() == ''){
$('#login_email').focus();
}
else if($('#login_password').val().trim() == ''){
$('#login_password').focus();
}
else {
$.ajax({
url:'login.php',
type:'post',
data: {
email: $('#login_email').val().trim(),
password: $('#login_password').val().trim()
},
success: function(data) {
if(data == 'Success'){
window.location = 'main.php';
}
else {
$('#login .ui-state-error p strong').html(data);
$('#login .ui-state-error').show();
}
}
});
}
});
});
HTML
<!DOCTYPE HTML>
<html lang="en">
<head>
<meta charset="utf-8">
<title>Social Networking</title>
<link rel="stylesheet" href="css/ui-lightness/jquery-ui-1.8.23.custom.css" type="text/css">
<link rel="stylesheet" href="css/reset.css" type="text/css">
<script src="js/jquery-1.8.0.min.js"></script>
<script src="js/jquery-ui-1.8.23.custom.min.js"></script>
<script src="js/login.js"></script>
</head>
<body>
<div id="wrapper">
<section>
<article>
<div id="tabs" class="tabs">
<ul>
<li><a href="#login">Login</a></li>
<li><a href="#register">Register</a></li>
</ul>
<div id="login">
<div class="ui-state-error ui-corner-all">
<p><span class="ui-icon ui-icon-alert"></span>
<strong></strong></p>
</div>
<label for="login_email">E-mail:</label>
<input id="login_email" type="email">
<label for="login_password">Password:</label>
<input id="login_password" type="password">
<label></label>
<button class='btnlogin'>Log in</button>
</div>
<div id="register">
<div class="ui-state-error ui-corner-all">
<p><span class="ui-icon ui-icon-alert"></span>
<strong></strong></p>
</div>
<label for="reg_name">Name:</label>
<input id="reg_name" type="text">
<label for="reg_email">E-mail:</label>
<input id="reg_email" type="email">
<label for="reg_password">Password:</label>
<input id="reg_password" type="password">
<label for="reg_cpassword">Confirm:</label>
<input id="reg_cpassword" type="password">
<label></label>
<button class='btnregister'>Register</button>
</div>
</div>
</article>
</section>
</div>
</body>
</html>