should change color on click