fixed setOnClicklistener