edit onclick working