Skip to content

feat: [키링 만들기] - 키링완성뷰 UI/UX 리팩토링 및 선물하기 기능 구현#33

Merged
giljihun merged 13 commits intodevelopfrom
feature/-키링-완성뷰-uiux
Jan 30, 2026

Hidden character warning

The head ref may contain hidden characters: "feature/-\ud0a4\ub9c1-\uc644\uc131\ubdf0-uiux"
Merged

feat: [키링 만들기] - 키링완성뷰 UI/UX 리팩토링 및 선물하기 기능 구현#33
giljihun merged 13 commits intodevelopfrom
feature/-키링-완성뷰-uiux

Conversation

@giljihun
Copy link
Member

@giljihun giljihun commented Jan 29, 2026 •
edited
Loading

🎯 PR 내용

키링완성뷰 리팩토링

  • 커스텀 네비게이션 → 기본 네비게이션 toolbar로 변경
  • KeyringSceneView ignoresSafeArea 제거로 뷰 계층 단순화 (이거 대체 왜 있던걸까?)
  • 액션 버튼 추상화 (이미지 저장, 공유, 선물하기)
  • 보관함 이동 버튼 추가 (trailing toolbar)
  • TabBarManager에 탭 전환 기능 추가

특히 툴바 변경으로, 가끔 발생하던 완성뷰 들어갈때 막 키링씬 흔들리던게 아에 사라진 걸로 보임.

Workshop 선물하기 기능 구현

  • KeyringPackageManager 생성 (Workshop/Collection 공용 포장 로직)

기존 포장 콜렉션뷰에 있던 로직 보고 추상화했습니다 리엘

  • CollectionViewModel이 KeyringPackageManager 사용하도록 리팩토링 (중복 코드 제거)
  • KeyringPackageCompleteView 추가 (Workshop용 포장완료 화면)
  • KeyringViewModelProtocol에 savedKeyringDocumentId 추가

버그 수정 및 개선

  • shareLink 직접 전달로 QR코드 안 보이던 타이밍 이슈 해결
  • 중복 포장 방지 (뒤로갔다 다시 선물하기 시 바로 포장완료뷰로 이동하도록 구현했음.)

이렇게 안하면 postOffice 무한 생성됨.

  • 포장 실패 시 네트워크 토스트 표시

📱 스크린샷 (UI 변경 시)

ScreenRecording_01-29-2026.14.1.mp4

++ 영상 이후
라이팅 변경
액션버튼 아이콘 변경
액션버튼 글래스 이펙트 변경

공유하기 구현 전에 피벗!

🔗 관련 이슈

✅ 체크리스트

  • 빌드 성공
  • 테스트 완료
  • Self-review 완료

giljihun added 11 commits January 29, 2026 10:33
@giljihun
refactor: 완성뷰 - 커스텀네비에서 기본네비
3a0fb15
@giljihun
refactor: KeyringSceneView - ignoreSafeArea 제거
3a1d160 
- 이게 왜 그동안 있었는지 모르겠습니다.
- 완성뷰에서 뷰를 그릴때 이 조건때문에 까다롭기만 했네요.
@giljihun
refactor: 완성뷰 - 커스텀네비 제거에 따른 뷰 계층 단순화
027b95a
@giljihun
feat: TabbarManager - 탭 전환 기능 추가
a15c4f7
@giljihun
style: 완성뷰 - 액션 버튼 추상화 및 하이파이 적용
0090d18
@giljihun
feat: KeyringViewModelProtocol에 savedKeyringDocumentId 추가
3c76bd0 
- 프로토콜에 저장된 키링 Document ID 프로퍼티 추가
- 6개 템플릿 VM에 프로퍼티 구현
- KeyringAdapter, WelcomeKeyringViewModel 프로토콜 준수
- Firebase 저장 후 Document ID 저장 로직 추가
@giljihun
feat: Workshop 선물하기 기능 구현
b96497a 
- KeyringPackageManager 생성 (공용 포장 로직)
- KeyringPackageCompleteView 추가 (Workshop용)
- WorkshopRoute에 packageComplete 라우트 추가
- WorkshopTab에 네비게이션 연결
@giljihun
refactor: CollectionViewModel이 KeyringPackageManager 사용하도록 변경
cd359dc 
- 중복된 포장 로직 제거 (~70줄)
- KeyringPackageManager.packageKeyring() 위임
@giljihun
fix: 선물 포장 시, QR코드가 안보이던 타이밍 이슈
10aa9f9 
- shareLink를 직접 전달해서 로딩 이슈 해결을 하고자함
@giljihun
feat: 키링완성뷰 -> 선물포장 -> 뒤로가기 -> 선물포장 -> 다시 포장 방지 구현
99ca716 
- 이미 포장된 경우 팝업 없이, 바로 포장완료뷰로 이동
- 포장 실패 네트워크 토스트도 추가
- KeyringViewModelProtocol에 packagedPostOfficeId, packagedShareLink 추가함
@giljihun
style: 키링완성to포장뷰 - 라이팅 재수정
7ea9e66
@giljihun giljihun self-assigned this Jan 29, 2026
@giljihun giljihun linked an issue Jan 29, 2026 that may be closed by this pull request
Feature: 키링 완성뷰 - UI/UX #32
Closed
jini-coding
jini-coding approved these changes Jan 30, 2026
View reviewed changes
Copy link
Member

@jini-coding jini-coding left a comment

Choose a reason for hiding this comment

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

그러게요 .ignoreSafeArea 요놈 왜 넣었었죠...ㅋㅋㅋ
개선된 점들이 많네요! 탭은 어떻게 잘 전환되려나 했는데 생각보다는 간단했군여...
개선이 필요하다 싶었던 부분들 잘 짚어주셔서 감사해요 리팩도, 타이밍 개선도 굿입니다.

@giljihun giljihun merged commit 2eebf78 into develop Jan 30, 2026
@giljihun giljihun deleted the feature/-키링-완성뷰-uiux branch January 30, 2026 02:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jini-coding jini-coding jini-coding approved these changes

Assignees

@giljihun giljihun

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Feature: 키링 완성뷰 - UI/UX

2 participants

@giljihun @jini-coding