try set to onclick instead