fix to onclick