키보드 단축키로 토픽을 추적하면 다른 토픽도 함께 추적됩니다

기여 버그
1

m t, m r 단축키를 사용하여 주제를 추적/추적 해제하면, 같은 창에서 최근 열람한 다른 주제들의 추적 상태도 함께 변경됩니다.

마우스를 사용할 때는 이러한 문제가 발생하지 않는 것 같습니다.

Firefox 68.4.1esr에서 확인했습니다. 처음에는 Rust 디스커스에서 발견했으며, 현재 이곳에서도 확인해 보았습니다.

5개의 좋아요
3

안녕하세요, 보고해 주셔서 감사합니다!

이 버그는 꽤 오래전부터 존재해 왔던 것 같습니다. 최근 변경 사항으로 인해 더 눈에 띄게 된 것일 수도 있겠네요.

요약하면 다음과 같습니다:

  • 변경 이벤트 수신 시 토픽 ID를 강제하지 않고 있습니다.
  • 단축키에 대한 스로틀링(throttling)을 적용하지 않아, ‘m’ 키를 누르고 있으면 동일한 이벤트가 무한정 전송됩니다…

다음 PR이 이 문제를 수정할 것입니다:

이번 주 안으로 병합(merge)될 것으로 예상됩니다.

7개의 좋아요
4

안녕하세요,
풀 리퀘스트가 병합된 시점부터 수정 사항이 배포되기까지 얼마나 걸리나요?
현재 여기서는 동작에 변화가 보이지 않습니다.

5

어떤 포럼인지에 따라 다릅니다. 해당 커밋은 master에 병합되고 stable 브랜치로 백포트되었으므로, 누구나 이를 받을 수 있어야 합니다.

4개의 좋아요
6

이 주제는 3일 후 자동으로 닫혔습니다. 더 이상 새 답변을 작성할 수 없습니다.

8

@Falco 이건 회귀인가요?

2개의 좋아요
9

안녕하세요, @david
이 discourse에서 여전히 문제를 재현할 수 있습니다. 그동안 재현 불가능했는지는 모르겠습니다.

2개의 좋아요
10

@joffreyjaffeux 한 번 더 확인해 주시겠어요?

12

@mpol 이 문제를 재현하지 못했습니다. meta에서 재현하기 위해 정확히 어떤 단계를 밟고 계신지 알려주실 수 있을까요?

1개의 좋아요
13

제 환경 쪽에 문제가 있었던 것이라면 죄송합니다. 하지만 방금 Firefox 68.6.0esr에서 확인해 보았습니다.

  1. 키보드 탐색만 사용하여, 새로 여는 탭에서 메인 페이지 https://meta.discourse.org/ 를 처음 열었을 때, 추적 중인 주제가 없습니다.
  2. ‘j’ 키를 누른 후 ‘Enter’ 키를 눌러 주제에 진입합니다.
  3. ‘j’ 키를 충분히 눌러 추천 주제까지 이동한 후, ‘Enter’ 키를 눌러 그중 하나에 진입합니다.
  4. ‘j’, ‘m’, ‘t’ 키를 누릅니다. 이제 이 주제와 메인 페이지에서 진입했던 주제 모두 추적되고 있습니다.

참고로, 3단계에서 'm’과 ‘t’ 키를 바로 눌렀을 때만 올바른 주제가 추적되었습니다. 하지만 그 후에도 ‘j’, ‘m’, ‘t’ 키를 누르면 이전 주제의 상태가 변경되었습니다.

1개의 좋아요
14

그래서 저는 이 특정 버그를 재현하지 못했고, @david도 마찬가지였습니다. 하지만 @david의 브라우저에서는 많은 요청이 동시에 진행 중이었고, 제 브라우저에서는 그렇지 않았습니다. 처음에는 브라우저 자체, 그리고 여러 브라우저가 열려 있는 것이 원인인 줄 알았습니다.

조사를 좀 해본 결과, 브라우저 크기(저에게는 좁았고, David에게는 넓었습니다)와 David에게는 표시되었지만 저에게는 표시되지 않던 주제 타임라인 버튼이 새로운 appEvents 핸들러를 계속 등록하고 있었기 때문임을 알게 되었습니다.

이것이 주제 추적 상황에서 상황을 크게 개선할 것이라고 생각합니다. 자세한 내용은 커밋에서 확인하실 수 있습니다:

4개의 좋아요
15

아, 그렇군요, 요청에 대해. 결국 위의 절차는 https://meta.discourse.org/t/146924/notifications로 동일한 POST 요청 두 개를 생성한 후, 문제의 원인이 될 것으로 추정되는 https://meta.discourse.org/t/113192/notifications로의 요청을 보낸 것이었습니다.

2개의 좋아요
16

네, 그런 문제들을 해결할 수 있을 거예요. 이제 머지니니 진행할 테니 곧 메타에서 테스트해 볼 수 있을 겁니다.

5개의 좋아요
17

@joffreyjaffeux 감사합니다. 몇 번 시도해 본 결과 여기서는 더 이상 문제를 재현할 수 없었고, 실제로 이제 /notifications로 보내는 요청이 정확히 하나만 보이며 중복 요청이나 다른 토픽에 대한 요청은 없습니다.

3개의 좋아요