Piccolo errore nella console dopo il clic su un video caricato

Bug
1

Correlato a:

Ecco un piccolo video che mostra il problema:

Sembra che la rimozione del listener di clic qui non funzioni. La prossima volta che si verifica un clic, il video-placeholder-container non esiste più:

https://github.com/discourse/discourse/blob/main/app/assets/javascripts/discourse/app/instance-initializers/video-placeholder.js#L16

Probabilmente a causa dell’uso di bind qui:

https://github.com/discourse/discourse/blob/main/app/assets/javascripts/discourse/app/instance-initializers/video-placeholder.js#L85

Forse una soluzione semplice sarebbe usare { once: true } in addEventListener invece?

3 Mi Piace
4

Grazie per la correzione @blake!

2 Mi Piace
5

Prego! Grazie per averlo segnalato e per il suggerimento :slight_smile:

3 Mi Piace