use appendchild instead of innerHTML