fixed onclick syntax