addEventListener  instead of onclick