Skip to content

[Feature] 플레이어 화면 전환 버튼 추가 및 오버레이 개선#99

Merged
jihyeonjjang merged 4 commits intodevfrom
feature/dismiss-fullscreen-btn
Jun 17, 2025
Merged

[Feature] 플레이어 화면 전환 버튼 추가 및 오버레이 개선#99
jihyeonjjang merged 4 commits intodevfrom
feature/dismiss-fullscreen-btn

Conversation

@jihyeonjjang
Copy link
Copy Markdown
Collaborator

변경 내용

  • 영상 재생이 끝난 후에도 오버레이가 계속 표시되도록 동작 수정
  • PlayerViewController에 닫기(Dismiss) 및 전체화면 전환 버튼 추가
  • FullScreenPlayerViewController에 전체화면 종료 버튼 추가

@jihyeonjjang jihyeonjjang self-assigned this Jun 16, 2025
@korbit-ai
Copy link
Copy Markdown

korbit-ai Bot commented Jun 16, 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.

@kdn0325 kdn0325 self-requested a review June 17, 2025 00:29
Copy link
Copy Markdown
Collaborator

@kdn0325 kdn0325 left a comment

Choose a reason for hiding this comment

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

@jihyeonjjang 컨플릭트 생겼습니다

@kdn0325 kdn0325 self-requested a review June 17, 2025 00:31
Comment thread PickaView/Views/Player/PlayerViewController.swift
Copy link
Copy Markdown
Collaborator

@kdn0325 kdn0325 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 04e4551 into dev Jun 17, 2025
@jihyeonjjang jihyeonjjang deleted the feature/dismiss-fullscreen-btn branch June 17, 2025 02:10
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