changes to onclick event