fix onclick event functinos