should be readonly until edit click