-
Notifications
You must be signed in to change notification settings - Fork 0
Feature&Chore/UI&Map: 크로스헤어 관련 기능 추가 및 오브젝트 배치 #94
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
kwkim526
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
확인했습니다.
엘리베이터 부분 코드를 보니 델리게이트 형식으로 바뀌었는데 어떤 차이점이 있는지 궁금합니다.
바뀐 내용 바탕으로 플레이어 사망 부분도 수정할 예정이라서 자세한 이유가 궁금합니다.
기존에 엘레베이터에서 직접 컴포넌트를 가져와서 함수를 호출하는 식으로 코드를 구성하게 되면 가독성이 매우 떨어지고 해당 클래스의 목적에 맞지 않다고 판단하여 델리게이트 형식으로 변경하기도 했고, 또 하나의 이유는 크로스헤어가 엘레베이터를 탑승하게 되면 안 보이게 처리해야 되는데 위와 같은 이유로 인해 델리게이트로 구현하는 것이 각 기능의 책임을 각 클래스에서 지게 끔 하였습니다. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
확인했습니다.
상호작용일 때 크로스 헤어의 불투명도를 60%정도로 낮춰도 좋을 것 같습니다.
bumyee11
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
확인했습니다.
정기 회의 14차-1차 PR (이상혁, UI & Map)
🎮 개발한 핵심 기능
🐛 고친 버그 & 코드 목록
📷 참고 자료
default.mp4
🔍 리뷰가 필요한 부분