update in click instead