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

[#29] CompletionCelebrationView 추가 #37

Merged
merged 6 commits into from
Nov 5, 2024

Conversation

zaehorang
Copy link
Collaborator

📝 작업 내용

이번 PR에서 작업한 내용을 간략히 설명해주세요

  • CompletionCelebrationView를 추가합니다.
  • CompletionCelebrationView와 CompletionReviewView를 연결합니다.
  • 공통적으로 쓰일 커스턴 네비게이션 백 버튼을 추가하, 스와이브로 백 동작을 가능하게 합니다.

스크린샷 (선택)

💬 리뷰 요구사항(선택)

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

  • ex) 메서드 XXX의 이름을 더 잘 짓고 싶은데 혹시 좋은 명칭이 있을까요?

@zaehorang zaehorang self-assigned this Nov 5, 2024
Copy link
Collaborator

@hyeonheebee hyeonheebee left a comment

Choose a reason for hiding this comment

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

고생하셨습니다! 좌 우 스와이프 통해서 화면 넘어가는 동작을 custom back button으로 구현해주신것으로 이해했는데 맞나요? :)

@@ -58,5 +58,5 @@ struct CompletionReflectionView: View {
}

#Preview {
CompletionReflectionView()
CompletionReviewView()
Copy link
Collaborator

Choose a reason for hiding this comment

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

훨씬 네이밍이 직관적이어 졌어요(제 기준) 감사합니다 !

- 일관된 네비게이션 UX 제공을 위해 ViewModifier로 커스텀 백 버튼 구현
- 인터랙티브 pop 제스처 활성화를 위헤 UINavigationController delegate 설정 수정 (좌측 스와이프 시 이전 화면으로 이동)
- CompletionCelebrationView를 구현하여 사용자가 완독 메시지 확인과 소감 작성 페이지로 이동 가능
Copy link
Collaborator

@hyeyeonie hyeyeonie left a comment

Choose a reason for hiding this comment

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

고생하셨습니다 ͙ᐟ

- 필요없는 주석을 제거합니다.
- 리베이스 과정에서 지워진 코드를 추가합니다. (네비게이션 바 백 버튼 관련 코드)
@zaehorang zaehorang force-pushed the feat/29--Completely-read-view branch from 4a7f4da to f4da778 Compare November 5, 2024 06:09
@zaehorang zaehorang merged commit 3d970de into develop Nov 5, 2024
@zaehorang zaehorang deleted the feat/29--Completely-read-view branch November 5, 2024 06:12
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.

3 participants