In onclick function, you need to change a bit. You have assignment '=' instead of comparison '==' in the if statement. Anyway, I also add to check if the username field is filled with only white spaces.
// original code
function make_blank() {
if(document.login.username.value ="Username"){
document.login.username.value ="";
}
}
// ===========
// suggested code
function make_blank() {
if((document.login.username.value.toLowerCase()=="username") { // if 'username', empty it
document.login.username.value = "";
}
else if (document.login.username.value.replace(/\s+/g, "")=="") { // if empty or white spaces, filled with 'username'
document.login.username.value = "Username";
}
} The function for password will always delete the existing password, do you intend it to be that way?