Skip to content

Conversation

@youn9k
Copy link
Member

@youn9k youn9k commented Dec 1, 2024

🤔 배경

서로 다른 순서라면 포즈를 취할 때 어색하기 때문에 순서를 맞춰주는 것이 중요했습니다.

📃 작업 내역

  • 참가자들 화면 순서 동기화
  • 닉네임 표시 (호스트는 Entity가 달라 임시로 설정해두었습니다.)
  • Toast 기존 토스트가 남아있으면 제거하고 새로운 Toast의 애니메이션을 시작합니다.

✅ 리뷰 노트

리뷰 해줘~~

🎨 스크린샷

🚀 테스트 방법

youn9k and others added 4 commits December 1, 2024 21:46
Co-Authored-By: Kiyoung <121777185+Kiyoung-Kim-57@users.noreply.github.com>
Co-Authored-By: Kiyoung <121777185+Kiyoung-Kim-57@users.noreply.github.com>
@youn9k youn9k added the ✨ feat 새로운 기능 추가 label Dec 1, 2024
@youn9k youn9k self-assigned this Dec 1, 2024
@youn9k youn9k requested review from 0Hooni, Kiyoung-Kim-57 and hsw1920 and removed request for Kiyoung-Kim-57 and hsw1920 December 1, 2024 15:24
@youn9k youn9k linked an issue Dec 1, 2024 that may be closed by this pull request
Copy link
Collaborator

@hsw1920 hsw1920 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

고생하셨습니당

불필요한 cancel 메소드 제거
Copy link
Collaborator

@0Hooni 0Hooni left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

수고 많으셨습니다!

Base automatically changed from fix/#145-remote-video-rendering to develop December 2, 2024 00:56
@youn9k youn9k merged commit 31cb0f8 into develop Dec 2, 2024
@youn9k youn9k deleted the feature/#151-participants-layout-sync branch December 2, 2024 00:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

✨ feat 새로운 기능 추가

Projects

None yet

Development

Successfully merging this pull request may close these issues.

방에 참가한 유저들의 화면 순서를 동기화합니다.

5 participants