fix onclick bug