rather, do it in the click handler