change on() to up()