replaced onclick scripts with attribs