jack2
12월 20, 2019, 11:05오전
1
재현 방법:
게시글 중 하나에 가서 편집(Edit)을 클릭하여 작성기를 엽니다. 아무것도 입력하지 마세요.
Esc 키를 누르면 => 예상대로 작성기가 완전히 사라집니다(게시글이 수정되지 않았으므로 최소화되지 않습니다).
브라우저 페이지를 새로고침하면 => 작성기가 다시 나타납니다.
2단계에서 Esc 키를 누르는 대신 취소(Cancel) 버튼을 클릭하면 동작이 정상적으로 이루어지며, 즉 브라우저 새로고침 후 작성기가 다시 나타나지 않습니다.
5개의 좋아요
@jack2
방금 해봤는데, 맞아요. 말씀하신 대로 정확히 작동합니다. 이게 버그인지, 원래 이렇게 동작하도록 설계된 건지 잘 모르겠네요.
3개의 좋아요
j.jaffeux
(Joffrey Jaffeux)
3월 29, 2020, 8:46오전
4
더 이상 재현이 안 되는데, try.discourse.org에서 아직 재현이 가능한가요?
jack2
3월 29, 2020, 9:32오전
5
네, try.discourse.org에서 방금 재현해 봤습니다.
잠깐! 무작위로 발생하네요! 가끔은 나타나지 않습니다.
j.jaffeux
(Joffrey Jaffeux)
3월 29, 2020, 9:34오전
6
일관된 재현 방법을 찾을 수 있나요? 수정에 대한 아이디어는 있지만, 일관된 재현 방법을 확보하는 것이 더 좋습니다. 부탁드립니다.
1개의 좋아요
j.jaffeux
(Joffrey Jaffeux)
3월 29, 2020, 9:39오전
8
좋아요! 제 수정 아이디어가 아마도 잘 작동할 것 같네요. 정말 감사드립니다
3개의 좋아요
j.jaffeux
(Joffrey Jaffeux)
3월 29, 2020, 12:39오후
9
예상보다 훨씬 더 까다로운 문제였습니다. 이 수정이 문제를 해결할 것 같습니다:
master ← jjaffeux:draft-escape-fix
closed 06:41AM - 30 Mar 20 UTC
This fix will also make sure the bootbox will be correctly shown when using esca… pe.
Test cases:
- edit post, wait for draft check, escape => should totally hide composer
- edit post, add a char, escape => should show bootbox, and reduce or hide depending on choice
Same behavior is expected when pressing on cancel.
여러 번 리뷰를 요청했으며, 병합까지 시간이 좀 걸릴 수 있습니다. 자세한 재현 방법을 공유해 주셔서 다시 한번 감사합니다
5개의 좋아요
j.jaffeux
(Joffrey Jaffeux)
3월 30, 2020, 6:44오전
10
간단한 업데이트입니다. 방금 PR을 닫았습니다. 수정은 잘 작동했지만, 더 높은 수준에서 해결하려고 합니다.
조금 시간이 걸릴 수도 있습니다
4개의 좋아요
sam
(Sam Saffron)
3월 31, 2020, 12:51오전
11
다음과 같이 수정했습니다:
committed 12:49AM - 31 Mar 20 UTC
When editing a post we were incorrectly saving a draft prior to user typing
Thi… s caused a bloat in the amount of drafts saved per user and inconsistency
around behavior of "escape" button.
It also lead to lots of warnings about draft conflicts when copying stuff
between posts.
The code is improved to use promises more appropriately, however further
changes are needed to clean up internals so methods consistently return
promises.
Too many methods in the controller sometimes return a promise and sometimes
an object. Long term the methods will become async and all of this will be
corrected.
이 파일은 정말 까다롭습니다 @eviltrout … async에 대한 지원이 되는 날을 손꼽아 기다리고 있어요. 많은 메서드들이 상황에 따라 프라미스를 반환하기도 하고 객체를 반환하기도 해서 로직을 파악하기가 매우 어렵거든요.
지금은 몇 가지 프라미스와 우회 처리를 도입해서, 편집 버튼을 클릭한 직후에 바로 초안을 저장하지 않도록 했습니다. 이 문제는 @supermathie도 정말 골치 아파했던 부분인데, 여러 개의 답변에서 텍스트를 복사해서 원글(OP)에 붙여넣는 작업을 매우 복잡하고 귀찮게 만들었거든요.
7개의 좋아요
이것은 제가 겪었던 많은 좌절감의 원인을 설명해 줍니다.
4개의 좋아요
j.jaffeux
(Joffrey Jaffeux)
에 닫힘
4월 4, 2020, 2:57오전
13
이 주제는 마지막 답변 후 4일 자동으로 닫혔습니다. 더 이상 새 답변을 작성할 수 없습니다.