Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remplace classList.replace pour éviter d'envoyer une erreur quand le navigateur ne supporte pas la méthode #1332

Conversation

marouria
Copy link
Contributor

@marouria marouria commented Feb 7, 2023

Erreur rollbar : TypeError: this.$refs.cercleBleu.classList.replace is not a function

La méthode classList.replace est normalement bien supportée sur tous les navigateurs dont Chrome Android
https://developer.mozilla.org/en-US/docs/Web/API/Element/classList

Du coup c'est peut-être le chargement de l'image qui a posé problème lors de l'utilisation de la méthode. Je propose donc de s'assurer qu'elle est présente pour lancer l'aimation.

@marouria
Copy link
Contributor Author

marouria commented Feb 7, 2023

@etienneCharignon
Copy link
Member

Ce que tu as changé me semble assez intéressant, il faut que je le relise plus en détail demain.

Par contre, je pense que ça ne va pas résoudre le problème.

La version du navigateur qui a causé l'erreur : Chrome 55 (sur Android 6.0)
D'après la doc que tu as mis en référence, replace n'apparaît sur chrome qu'à partir de la version 61
À mon avis, c'est bien ça le problème.

De plus, l'erreur le dit clairement : this.$refs.cercleBleu.classList.replace is not a function

C'est la deuxième fois qu'on tombe sur ce problème en fait. Ça ne me paraît pas forcément d'un intérêt énorme de chercher à supporter de si vieux navigateurs. Cependant, pour éviter l'erreur, ce n'est peut-être pas grand-chose de faire le replace à la main, voir de la faire un utilisant vuejs.

Je me demande si nous avons d'autres usages de classList.replace ?

@etienneCharignon
Copy link
Member

J'ai proposé une solution vuejs

@etienneCharignon
Copy link
Member

Il reste un "replace" dans… l'inventaire !

@marouria marouria changed the title n'envoie pas d'erreur si l'image du cercle bleu n'a pas eu le temps de charger Remplace classList.replace pour éviter d'envoyer une erreur quand le navigateur ne supporte pas la méthode Feb 8, 2023
@etienneCharignon etienneCharignon merged commit e47c532 into develop Feb 8, 2023
@etienneCharignon etienneCharignon deleted the 1652-🐛-erreur-rollbar-702-à-707-sur-le-cercle-bleu-du-dragdrop branch February 8, 2023 15:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants