inline javascript is a bad practice in this case