changed onclick to function and not text