Rastrear um tópico usando o atalho de teclado rastreia outros tópicos também

Contribuir Bug
1

Quando uso os atalhos m t e m r para acompanhar/desacompanhar um tópico, isso também altera o status de acompanhamento de outros tópicos que visualizei recentemente na mesma janela.

Isso não parece acontecer quando uso o mouse.

Observado no Firefox 68.4.1esr. Percebi isso pela primeira vez no Discourse do Rust, e agora verifiquei também aqui.

5 curtidas
3

Olá, obrigado pelo relatório!

Acho que esse bug existe há bastante tempo; talvez as mudanças recentes o tenham tornado mais visível.

Resumindo:

  • não estamos validando o ID do tópico ao receber o evento de alteração
  • não estamos limitando (throttling) o atalho, o que significa que, se você segurar a tecla “m”, o mesmo evento será enviado indefinidamente…

Isso deve corrigir o problema:

Isso deve ser mesclado até o final da semana.

7 curtidas
4

Olá,
Quanto tempo deve levar desde a mesclagem do pull request até a correção ser implantada?
No momento, não vejo nenhuma mudança de comportamento aqui.

5

Depende de qual fórum. O commit foi mesclado na master e portado de volta para a branch estável, então todos devem conseguir obtê-lo.

4 curtidas
6

Este tópico foi automaticamente fechado após 3 dias. Novas respostas não são mais permitidas.

8

@Falco isso regrediu?

2 curtidas
9

Olá @david
Ainda consigo reproduzir o problema neste fórum. Não sei se ele se tornou irreproduzível no intervalo.

2 curtidas
10

@joffreyjaffeux você poderia dar outra olhada, por favor?

12

@mpol não conseguimos reproduzir isso. Você poderia nos explicar os passos exatos que você segue para reproduzir isso aqui no Meta?

1 curtida
13

Peço desculpas se o problema estiver do meu lado, mas aqui está: no Firefox 68.6.0esr, agora mesmo.

  1. Usando apenas a navegação por teclado, começando em uma nova aba com a página principal recém-aberta https://meta.discourse.org/, não tenho nenhum tópico acompanhado no início.
  2. Pressiono ‘j’, depois ‘Enter’ para entrar em um tópico.
  3. Pressiono ‘j’ várias vezes até chegar aos tópicos sugeridos e pressionar ‘Enter’ para entrar em um deles.
  4. Pressiono ‘j’, ‘m’, ‘t’ — agora estou acompanhando tanto este tópico quanto o acessado a partir da página principal.

Aliás, acabei de perceber que, ao pressionar ‘m’ e ‘t’ imediatamente no passo 3, apenas o tópico correto foi acompanhado. Mas mesmo após isso, ao pressionar ‘j’, ‘m’, ‘t’, o estado do tópico anterior foi alterado.

1 curtida
14

Então, não consegui reproduzir exatamente esse bug, e o @david também não. No entanto, o @david tinha muitas requisições em andamento no navegador, o que eu não tinha. Minha primeira ideia foi que o problema estava no navegador e, em seguida, em múltiplos navegadores abertos?

Após algumas pesquisas, entendi que o problema vinha do tamanho do navegador (que era estreito para mim e grande para o David) e do botão de linha do tempo do tópico, que continuava registrando novos manipuladores de appEvents, sendo exibido para o David e não para mim.

Acho que isso deve melhorar bastante a situação no rastreamento de tópicos. Mais detalhes no commit:

4 curtidas
15

Ah, sim, solicitações. Então, no final, o procedimento acima gerou duas POSTs idênticas para https://meta.discourse.org/t/146924/notifications e, em seguida, a que acredito ser problemática, para https://meta.discourse.org/t/113192/notifications.

2 curtidas
16

Sim, isso deve resolver esse tipo de coisa. Estou mesclando agora para que você possa testar no Meta em breve.

5 curtidas
17

@joffreyjaffeux Obrigado. Após algumas tentativas, não consegui mais reproduzir o problema aqui e, de fato, agora vejo apenas uma única requisição correta para /notifications — sem duplicatas e sem requisições para outros tópicos.

3 curtidas