added proper click listener