wan2land / unchartedwaters2

대항해시대2를 웹으로 편하게 즐기자!
https://water2.dist.be
365 stars 88 forks source link

# 가상키보드 연속 입력 개선 #10

Closed dorian-kwon closed 3 years ago

dorian-kwon commented 3 years ago

가상키보드 연속 입력을 개선해보았어요!

wan2land commented 3 years ago

PR 고맙습니다! 😄 

코드리뷰 해본 결과, 다음과 같은 문제가 확인되었습니다.

1. Click 이벤트를 mousedown과 mouseup으로 분리시 드래그에 대해서 대응해야합니다

예를들어, A버튼에서 mousedown후, 그대로 이동해서 B버튼에서 mouseup을 하는 경우 이벤트는 다음과 같이 입력됩니다.

그렇게 되면 mouseup A 이벤트가 캐치가 안되어서 계속해서 A버튼을 누른상태가 되어버립니다.
mouseleave, touchleave이벤트에 대한 처리가 추가될 필요성이 있어보입니다.

2. Key는 Vue Component라서 이벤트를 Key Component로 전달할 때 4개로 전달할 필요가 없어보입니다.
mousedown과 touchstart는 keydown(컴포넌트 이벤트) 이벤트로 묶고, mouseup, touchend는 keyup 이벤트로 묶어야 할 것 같습니다. :-)

dorian-kwon commented 3 years ago

피드백 감사합니다.

잠깐 짬내서 연속 터치만 추가한거라 많이 부족했네요 ㅠㅠ

말씀 주신 부분 수정했습니다.

피드백 부탁드려요

wan2land commented 3 years ago

멀티터치 부분을 조금 손봐야할 것 같은데, 여기서부턴 제거 고쳐서 올려놓을게요! 고맙습니다! :-)