oops didn't do onclick handler correctly