Suivre un sujet avec le raccourci clavier suit aussi d'autres sujets

Contribuer Bug
1

Lorsque j’utilise les raccourcis m t et m r pour suivre/ne plus suivre un sujet, cela modifie également le statut de suivi d’autres sujets que j’ai consultés récemment dans la même fenêtre.

Cela ne semble pas se produire lors de l’utilisation de la souris.

Observé avec Firefox 68.4.1esr. J’ai remarqué ce problème en premier sur le forum Rust, et je l’ai maintenant vérifié ici également.

5 « J'aime »
3

Salut, merci pour le rapport !

Je pense que ce bug existe depuis un certain temps, peut-être que des changements récents l’ont rendu plus visible.

Pour faire court :

  • nous n’imposons pas l’ID du sujet lors de la réception de l’événement de changement
  • nous ne limitons pas le raccourci clavier, ce qui signifie que si vous maintenez la touche “m” enfoncée, le même événement sera envoyé indéfiniment…

Cela devrait régler le problème :

Cela devrait être fusionné d’ici la fin de la semaine.

7 « J'aime »
4

Bonjour,
Combien de temps faut-il normalement entre l’acceptation de la pull request et le déploiement de la correction ?
Pour le moment, je ne constate aucun changement de comportement ici.

5

Cela dépend du forum. Le commit a été fusionné dans master et rétroporté dans la branche stable, donc tout le monde devrait pouvoir l’obtenir.

4 « J'aime »
6

Ce sujet a été automatiquement fermé après 3 jours. De nouvelles réponses ne sont plus autorisées.

8

@Falco, cela a-t-il régressé ?

2 « J'aime »
9

Bonjour @david
Je parviens toujours à reproduire le problème sur ce forum. Je ne sais pas s’il était impossible à reproduire entre-temps.

2 « J'aime »
10

@joffreyjaffeux pourrais-tu jeter un autre coup d’œil, s’il te plaît ?

12

@mpol nous n’avons pas réussi à reproduire ce problème. Pourriez-vous nous décrire les étapes exactes que vous suivez pour le reproduire ici sur Meta ?

1 « J'aime »
13

Désolé si le problème vient de mon côté, mais voici ce qui se passe avec Firefox 68.6.0esr à l’instant.

  1. En utilisant uniquement la navigation au clavier, à partir d’une page principale fraîchement ouverte (https://meta.discourse.org/) dans un nouvel onglet, je n’ai aucun sujet suivi au départ.
  2. J’appuie sur ‘j’, puis sur ‘Entrée’ pour entrer dans un sujet.
  3. J’appuie suffisamment de fois sur ‘j’ pour atteindre les sujets suggérés, puis j’appuie sur ‘Entrée’ pour en choisir un.
  4. J’appuie sur ‘j’, ‘m’, ‘t’ — maintenant, je suis à la fois ce sujet et celui ouvert depuis la page principale.

Au fait, je viens de remarquer que lorsque j’ai appuyé sur ‘m’, ‘t’ immédiatement à l’étape 3, seul le sujet correct était suivi. Mais même après cela, appuyer sur ‘j’, ‘m’, ‘t’ a modifié l’état du sujet précédent.

1 « J'aime »
14

Je n’ai donc pas pu reproduire ce bug exact, et @david non plus, mais @david avait de nombreuses requêtes en cours dans son navigateur, ce que je n’avais pas. Ma première idée était le navigateur, puis plusieurs navigateurs ouverts ?

Après quelques recherches, j’ai compris que cela provenait de la taille du navigateur (qui était étroite pour moi et large pour David) et du bouton de chronologie du sujet qui continuait d’enregistrer de nouveaux gestionnaires d’appEvents, affiché pour David mais pas pour moi.

Je pense que cela devrait grandement améliorer la situation concernant le suivi des sujets, plus de détails dans le commit :

4 « J'aime »
15

Ah, oui, les requêtes. Ainsi, à la fin, la procédure ci-dessus a généré deux requêtes POST identiques vers https://meta.discourse.org/t/146924/notifications, puis celle qui semble poser problème, vers https://meta.discourse.org/t/113192/notifications.

2 « J'aime »
16

Oui, cela devrait régler ce genre de problèmes. Je le fusionne maintenant, vous pourrez donc le tester sur Meta bientôt.

5 « J'aime »
17

@joffreyjaffeux Merci. Après quelques tentatives, je ne parviens plus à reproduire le problème ici et je vois maintenant toujours une seule et bonne requête vers /notifications — pas de doublons et aucune requête pour d’autres sujets.

3 « J'aime »