made code more semantic
removed name= because I misunderstood its use and it is not needed.
fixed some indent issues
changed a couple buttons to <a> to meet requirments