Click-listeners are added multiple times to enhanced blocks

[derpondus]

Description

It goes for the timestamp and the spoiler block:

This is a problem if an even number of listeners is fired, as all of them are bubbling and the state is not changed.

TODO

Figure out what causes this

Can I work on this?

• Yes

[martian0x80]

I can only recreate this by navigating back and forth between pages on Crunchyroll.

I can't recreate this after adding a simple check to prevent adding multiple listeners.

comment.querySelectorAll("span.crunchy-comments-spoiler-block").forEach((spoiler) => {
	if (spoiler.dataset.hasEventListener === "true") return
	spoiler.addEventListener("click", () => {
		spoiler.classList.toggle("revealed")
	})
	spoiler.dataset.hasEventListener = "true"
})

Can you test if this works?

[derpondus]

I am not able to reproduce it anymore :D

Seems to be fixed 👍

ps.: sorry this took me so long, I was ill and then swamped with work afterward 😅

[derpondus]

Oh no, I just encountered a case in which the eventlistener was missing entirely...

(no "event" tag next to the element)

I don't remember what exactly I did, but the has-event-listener value is true, so something can become inconsistent with this fix.
(I believe, I just clicked on continue to watch on the main page, but there has to be something else to it.)