Skip to content

[Feature] Player 화면 구성 및 기능 구현#80

Merged
jihyeonjjang merged 7 commits intodevfrom
feature/player-ui-collectionview-refactor
Jun 16, 2025
Merged

[Feature] Player 화면 구성 및 기능 구현#80
jihyeonjjang merged 7 commits intodevfrom
feature/player-ui-collectionview-refactor

Conversation

@jihyeonjjang
Copy link
Copy Markdown
Collaborator

변경 내용

  • PlayerViewController 기본 UI 구성
    • 스토리보드 videoPlayerView에 videoContainerView 추가
  • Home → PlayerViewController로 CoreDataManager 의존성 주입
  • PlayerView에서 셀 선택 시 새 비디오로 화면 전환 기능 추가
  • 좋아요 기능 구현 (CoreData 상태 업데이트 및 UI 반영)
  • 다크모드 대응 및 UI 요소 스타일 개선

- UICollectionView 기반 셀 및 레이아웃 설정
- 영상 정보를 표시하는 Header 뷰 구성
- 수평 스크롤 셀(HorizontalCollectionViewCell) 및 TagCollectionView 구현
- PlayerViewCollectionViewCell 내부 레이아웃 및 콘텐츠 설정
- 셀 선택 시 해당 비디오를 기반으로 새로운 PlayerViewController를 띄움
- 기존 화면은 닫고, 새로 생성된 ViewModel과 함께 전환 처리
- 다크모드 색상 추가
- placeholder 색상 변경
- 썸네일 이미지 및 contentMode 개선
- 태그 cornerRadius 설정
@korbit-ai
Copy link
Copy Markdown

korbit-ai Bot commented Jun 15, 2025

You've used up your 5 PR reviews for this month under the Korbit Starter Plan. You'll get 5 more reviews on June 22nd, 2025 or you can upgrade to Pro for unlimited PR reviews and enhanced features in your Korbit Console.

Copy link
Copy Markdown
Owner

@TeddKo TeddKo 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
Copy Markdown
Collaborator

@vinyl-nyl vinyl-nyl 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
Copy Markdown
Collaborator

@giseungNoh giseungNoh left a comment

Choose a reason for hiding this comment

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

확인했습니다.

@jihyeonjjang jihyeonjjang merged commit 851f794 into dev Jun 16, 2025
@jihyeonjjang jihyeonjjang deleted the feature/player-ui-collectionview-refactor branch June 16, 2025 04:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants