using on instead of bind and click