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

Fix lesechos #242

Merged
merged 2 commits into from
Sep 1, 2024
Merged

Fix lesechos #242

merged 2 commits into from
Sep 1, 2024

Conversation

Write
Copy link
Collaborator

@Write Write commented Aug 31, 2024

This fix #241

Copy link
Owner

@lovasoa lovasoa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Merci!

@lovasoa lovasoa merged commit f2b2085 into lovasoa:master Sep 1, 2024
1 check passed
@Altonss
Copy link
Contributor

Altonss commented Sep 1, 2024

Thanks for the fix! Sometimes for a page that is initially loaded, the button still does not appear. I need to manually reload the page several times for the button to appear 🤔

@Write
Copy link
Collaborator Author

Write commented Sep 1, 2024

Thanks for the fix! Sometimes for a page that is initially loaded, the button still does not appear. I need to manually reload the page several times for the button to appear 🤔

Which browser ? So I can do more testing.

@Altonss
Copy link
Contributor

Altonss commented Sep 1, 2024

Firefox 129.0.2

@Write
Copy link
Collaborator Author

Write commented Sep 1, 2024

Firefox 129.0.2

Tu peux me confirmer si le bug n'apparaît que quand :

  • Tu navigues de pages en pages ?
    Ou bien
  • Uniquement quand tu vas sur l'article via l'URL directement ?

Pour le moment, je n'ai pas réussi à reproduire le bug, j'ai le bouton qui est apparu deux fois (ça m'est arrivé la toute première fois) mais c'est tout.

@Altonss
Copy link
Contributor

Altonss commented Sep 1, 2024

Uniquement quand tu vas sur l'article via l'URL directement ?

Ce bug n'apparaît que que je vais sur l'article via l'URL direcement. Je viens de remarque que le fait de désactiver une autre extension (NoScript) semblait régler le souci. Mais l'utilisation de cette extension ne devrait pas poser de problème.

De pages en pages, je viens de remarquer de nouveau le bug du double bouton europresse 🤔

@Write
Copy link
Collaborator Author

Write commented Sep 1, 2024

Uniquement quand tu vas sur l'article via l'URL directement ?

Ce bug n'apparaît que que je vais sur l'article via l'URL direcement. Je viens de remarque que le fait de désactiver une autre extension (NoScript) semblait régler le souci. Mais l'utilisation de cette extension ne devrait pas poser de problème.

De pages en pages, je viens de remarquer de nouveau le bug du double bouton europresse 🤔

De mémoire NoScript bloque le JS des pages à 100% donc ça ne m'étonne pas que des bugs existent avec cette extension.

Il faut vraiment que je trouve un moyen de régler la race condition des deux boutons du coup.
Si ça marche sans NoScript pour le reste c'eest déjà top.

@Altonss
Copy link
Contributor

Altonss commented Sep 1, 2024

De mémoire NoScript bloque le JS des pages à 100% donc ça ne m'étonne pas que des bugs existent avec cette extension.

J'autorise le nécessaire, et la page s'affiche parfaitement. De page en page le bouton s'affiche correcement, donc c'est la configuration d'ophirofox qui ne semble pas "stable" sur ce site. Sur d'autres sites je n'ai aucun souci d'interférence entre NoScript et Ophirofox.

La configuration pour les échos semble particulièrement complexe, peut-être qu'une simplification résoudrait les deux soucis à la fois? :)

@Write
Copy link
Collaborator Author

Write commented Sep 1, 2024

De mémoire NoScript bloque le JS des pages à 100% donc ça ne m'étonne pas que des bugs existent avec cette extension.

J'autorise le nécessaire, et la page s'affiche parfaitement. De page en page le bouton s'affiche correcement, donc c'est la configuration d'ophirofox qui ne semble pas "stable" sur ce site. Sur d'autres sites je n'ai aucun souci d'interférence entre NoScript et Ophirofox.

La configuration pour les échos semble particulièrement complexe, peut-être qu'une simplification résoudrait les deux soucis à la fois? :)

Comme tu le dis leur site est complexe, ce qui fait que je ne pense pas que je vais me pencher sur le support de NoScript en plus de cela :/ le risque d'effets de bord indésirable serait accru.

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.

Les échos: bug bouton "Lire sur Europresse"
3 participants