onclick handler must be set as attribute