fixed onclick listener login