boostcamp-2020 / Project18-B-Web-Duxit

🎲 야찌⛵️ 팀의 웹 기반 스토리텔링 보드게임 <Duxit> 🐤
https://duxit.ga
66 stars 7 forks source link

어쩌면 대대적인 리팩토링 + Game화면 파비콘 추가 #147

Closed sbyeol3 closed 3 years ago

sbyeol3 commented 3 years ago

💁 설명

기존 guesser select card와 teller select card 씬 2개 모두 카드 클릭 시 모달이 띄워지는 로직이 있었는데 로직이 비슷하다보니 굉장히 유사한 20줄의 코드가 중복적으로 들어있는 문제가 있었습니다. -> 이 함수를 유틸로 빼고 파라미터로 onClickCard 함수를 실행하게끔 바꾸었습니다.

해람님이 유틸쪽이 커지다 보니 타입과 유틸을 구분했으면 좋겠다는 리뷰를 주셔서 CONST하게 쓰기 위해 선언한 유틸들을 타입 디렉토리로 넣고 이전 코드 경로를 모두 수정했습니다. (일단 제가 다 테스트 해봤는데 정상적으로 작동하는데 놓친 부분이 있을 수 있습니다.)

HTML 엔트리가 2개라서 파비콘을 두 엔트리 모두에 적용을 해야 하는데 제가 main에만 넣어두었더군요.. game에서도 보이게끔 파비콘 옵션 추가했습니다.

📑 체크리스트

구현한 목록 체크리스트

🚧 주의 사항

PR을 읽을 때 살펴볼 사항

코드 자체는 거의 동일합니다..! 변경사항은 많을지라도 이게 파일을 옮기다보니...😅