카드 매니저가 선택된 카드의 아이디를 알고 있으므로 카드 매니저에서 해당 효과를 구현하게 했습니다.
애니메이션 효과가 들어가다보니 이미 생성되고 사라지고 있는 와중에 다음 씬이 렌더되는 버그가 있었습니다.
그래서 각 씬마다 wrapupInterval 변수를 둘 수 있는데 이 변수는 wrapup이 되고 다음 씬이 render 되기 전까지의 딜레이를 의미합니다.
그러다보니.. 시간이 안맞게 되어서 (progressbar) 나중에 랜덤 선택 기능을 추가 구현하면서 서버가 아예 픽스된 시간을 주는 게 더 좋을 듯 합니다.
아 추가로 다음 씬의 클래스를 인스턴스로 만들 때 이미 생성자의 코드는 실행되므로 렌더가 겹치게 되는데요.
그래서 생성자에 있던 렌더 관련 함수는 렌더 쪽으로 뺐습니다! (@ramram1048 )
애니메이션을 잘 못해서.. 좀 부자연스럽기도 합니다.
후보 1
후보 2
후보 3
추후 랜덤선택 기능을 하게 된다면, 텔러가 토픽을 알 수 있어야 하므로 토픽도 보이게 했습니다.
close #150
📑 체크리스트
구현한 목록 체크리스트
[ ] 텔러가 카드를 선택하고 player waiting 씬으로 넘어가기 전에 선택한 카드가 올라가는 효과를 보여줍니다.
🚧 주의 사항
PR을 읽을 때 살펴볼 사항
PR을 빈번하게 하여.. 죄송합니다만 모두에게 좋자고 하는 거니.. 양해 부터ㅏㄱ... 드립니다...
gif에는 플레이어가 2명인데 제가 귀찮아서.. 넘버를 잠깐 수정한 것입니다. 버그가 아닙니다.
일부 리팩토링 있음
크롱님이 예전에 addClass가 한번에 여러 클래스를 받을 수 있으면 좋지 않겠냐는 리뷰를 남겨주셨는데요.
그래서 addClass가 배열을 받을 수 있게 했습니다.
다만 다른 코드에 영향을 주지 않기 위해 배열일 때만 forEach로 돌고 그 외에는 그냥 추가해줍니다.
💁 설명
카드 매니저가 선택된 카드의 아이디를 알고 있으므로 카드 매니저에서 해당 효과를 구현하게 했습니다. 애니메이션 효과가 들어가다보니 이미 생성되고 사라지고 있는 와중에 다음 씬이 렌더되는 버그가 있었습니다. 그래서 각 씬마다
wrapupInterval변수를 둘 수 있는데 이 변수는 wrapup이 되고 다음 씬이 render 되기 전까지의 딜레이를 의미합니다. 그러다보니.. 시간이 안맞게 되어서 (progressbar) 나중에 랜덤 선택 기능을 추가 구현하면서 서버가 아예 픽스된 시간을 주는 게 더 좋을 듯 합니다.아 추가로 다음 씬의 클래스를 인스턴스로 만들 때 이미 생성자의 코드는 실행되므로 렌더가 겹치게 되는데요. 그래서 생성자에 있던 렌더 관련 함수는 렌더 쪽으로 뺐습니다! (@ramram1048 )
애니메이션을 잘 못해서.. 좀 부자연스럽기도 합니다.
후보 1
후보 2
후보 3
close #150
📑 체크리스트
🚧 주의 사항
일부 리팩토링 있음
addClass가 한번에 여러 클래스를 받을 수 있으면 좋지 않겠냐는 리뷰를 남겨주셨는데요.