События предпочтительнее навешивать через addEventListener. В любом случае точка с запятой не нужна