was missing the onclick parameter