site dev help megathread

Open and close the report modal multiple times and check the event listeners—every time the modal is opened, an event listener on the submit button is re-added. You can either directly set the onclick property of the button to make sure that there is only one custom event listener (easy, but kinda awful), or you can remove the event listener every time the modal closes (better approach, use the same thing on line 8 in comments_v.js to get the instance and add the corresponding removeEventListener).

throttling submissions would probably be the best way of fixing this, would also prevent people from actually double clicking it. hmm looks like postToast only has a callback for success right? ideally you'd handle enabling it on any response, so i'd probably just patch this with a timer to prevent double submits - either with a variable like this or by disabling the button.

edit: oh didn't realize this was all on window, can probably chuck the canSubmit on reportCommentButton.canSubmit = true in that case

let canSubmit = true;
reportCommentButton.addEventListener('click', function() {
	if (!canSubmit) return;
	canSubmit = false;
	setTimeout( function() {
		canSubmit = true;
	}, 1000 );

	this.innerHTML='Reporting comment';

	postToast(this, `/report/comment/${id}`,
		{
			"reason": reason_comment.value
		},
		() => { canSubmit = true }
	);
})