Skip to content
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

[Feature/#66] 영상 타임라인을 나타낼 수 있는 UI #67

Merged
merged 10 commits into from
Nov 29, 2024

Conversation

around-forest
Copy link
Collaborator

@around-forest around-forest commented Nov 28, 2024

관련 이슈

✅ 완료 및 수정 내역

  • 공유된 영상의 타임라인을 나타낼 수 있는 UI

🛠️ 테스트 방법

  • 아직 테스트 코드가 없습니다. 지혜님의 PR 머지 후 테스트 하거나, ConnectionViewController의 nextButton에 해당 VC를 연결하여 테스트할 수 있습니다.

📝 리뷰 노트

CollectionView

해당 테스크를 지혜님이 이어서 Long-Pressed 순서 변경을 구현해야합니다.
이 과정에서 스타일을 최대한 맞추기 위해 지혜님이 VideoListViewController에서 구현하셨던 CollectionView의 스타일을 따라 구현했습니다.

기타 수정 사항

  • 윤회님이 초기로 구현하셨던 VC의 네이밍등을 파일 명과 동일하게 수정했습니다.
  • VideoListViewController의 nextButton에 해당 VC로의 연결 코드를 수정 및 추가했습니다.
  • DIContainer에 VM을 등록했는데, 추후 usecase 연결이 필요합니다.
  • 기존 ConnectionView의 nextButton UI를 VideoListView에 있는 nextButton과 동일하도록 수정했습니다.

📱 스크린샷(선택)

image

@around-forest around-forest self-assigned this Nov 28, 2024
@around-forest around-forest linked an issue Nov 28, 2024 that may be closed by this pull request
1 task
Copy link
Collaborator

@jungseokyoung-cloud jungseokyoung-cloud left a comment

Choose a reason for hiding this comment

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

고생하셨습니다! 건우님 혹시 영상 재생 화면과 타임 라인 사이에 버튼과 여러가지들이 들어갈 예정인데, 혹시 80픽셀 정도 띄어 주실 수 있을까요?

@around-forest
Copy link
Collaborator Author

적용했습니다!
일단 top Padding에 80을 추가했는데, 차라리 나중에 top Padding의 기준을 영상 재생 화면과 타임라인 사이에 들어가는 요소로 수정하는게 보다 나을 듯 합니다!

Copy link
Collaborator

@051198Hz 051198Hz left a comment

Choose a reason for hiding this comment

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

고생 많으셨습니다~ !!
변경이 필요한 사항은 보이지않아 승인드립니다

Copy link
Collaborator

@LURKS02 LURKS02 left a comment

Choose a reason for hiding this comment

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

고생하셨습니다! PR 승인해두었습니다.

@around-forest around-forest merged commit 2bfa3a8 into develop Nov 29, 2024
3 checks passed
@around-forest around-forest deleted the feature/#66 branch November 29, 2024 14:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

영상 타임라인을 나타낼 수 있는 UI
4 participants