buttons should have type=button