fixed onclicklistener