Les Echos : Enhance error handling (querySelector). Refactor MutationObservers.

lovasoa / ophirofox

Une extension pour navigateur qui permet de lire les articles de presse en ligne sur le compte de bibliothèques ayant souscrit à europresse

https://ophirofox.ophir.dev

Mozilla Public License 2.0

63 stars 164 forks source link

Les Echos : Enhance error handling (querySelector). Refactor MutationObservers. #243

Closed Write closed 2 months ago

Write commented 2 months ago

This is a complete refactor that enhance robustness for LesEchos. Tested on Firefox / Chrome. Race condition where two buttons were added seems fixed too.

Reduce querySelector usage by passing metaElement to isPremium function.
Fix race condition.
Correctly disconnect observer.

Altonss commented 2 months ago

This is a complete refactor that enhance robustness for LesEchos. Tested on Firefox / Chrome. Race condition where two buttons were added seems fixed too.
* Reduce querySelector usage by passing metaElement to isPremium function.

* Fix race condition.

* Correctly disconnect observer.

Issues doesn't seem fixed on firefox (tried latest commit of the PR):

the double button still happens when navigating using page to page
button sometimes doesn't appear when opening single article individually (not related to NoScript!)

Write commented 2 months ago

Je confirme pour le double boutons... par contre cela ne m'arrive que la première fois (au démarrage de l'extension).

Je n'ai pas réussi à reproduire le problème du bouton qui ne s'affiche pas du tout en ouvrant un article seul.

Write commented 2 months ago

J'ai pris le taureau par les cornes. Normalement, c'est bon pour le double bouton. Pour l'autre bug, je n'arrive pas à reproduire.

Write commented 2 months ago

~~Maintenant sur Firefox il y a tout le temps deux fois le bouton. Sur Chrome c'est ok, mais très rarement le bouton n'apparait en effet pas du tout.~~

~~Je suis vraiment désolé,~~

~~Je proposerai un nouveau PR quand j'aurai vraiment testé TOUS les cas de figures sur TOUS les navigateurs de fond en comble.~~

Ok, je n'ai rien dit, c'est juste que j'avais l'userscript qui tournait en même temps sur Firefox.

Le double bouton devrait être ok, cependant sur Chrome de temps à autre en effet le bouton n'apparaît pas. Peux être sur Firefox aussi, mais pas réussi à reproduire encore.