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

Réparation du bouton Libération #188

Merged
merged 9 commits into from
Jan 18, 2024
Merged

Conversation

fmoncomble
Copy link
Contributor

Bonjour,

Le bouton n'apparaissait plus sur les articles de Libé.

@lovasoa lovasoa merged commit 249d5eb into lovasoa:master Jan 18, 2024
1 check passed
@fmoncomble
Copy link
Contributor Author

Ça n'invalide pas mes modifications, mais à y mieux regarder le bouton n'avait pas "disparu" comme suite à une modification de la structure de la page, mais nécessite un voire plusieurs rechargements… Ce que ma modification n'a malheureusement pas réglé… Si quelqu'un a une idée du pourquoi…

@lnoss
Copy link
Contributor

lnoss commented Jan 26, 2024

L'explication la plus simple : l'extension qu'elle s'exécute avant ou après est généralement plus rapide à terminer sa tâche que la bibliothèque JavaScript utilisée pour l'interface de Libération.

Deux solutions simples :

  • Tenter de détecter quand l'interface est complètement chargée. Comme souvent, un indicateur est ajoutée sur l'élément body ou le main. C'est toujours mystifier, mais on peut surveiller les mutations du DOM pour savoir quand une classe est ajoutée, puis ajouter le bouton ;
  • Ajouter le bouton de l'extension comme un bourrin et regarder s'il y a une mutation du DOM qui la retire. Rajouter le bouton à nouveau.

Sûrement d'autres solutions, sachant que ces solutions simples s'appuient sur des observateurs des mutations du DOM, assez lourd et qui doivent être désactivés quand plus nécessaires.

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.

3 participants