Skip to content

fix: 홈, 설정 QA 반영#299

Merged
jihun32 merged 5 commits into
developfrom
fix/#298/TWI-84
May 14, 2026
Merged

fix: 홈, 설정 QA 반영#299
jihun32 merged 5 commits into
developfrom
fix/#298/TWI-84

Conversation

@clxxrlove
Copy link
Copy Markdown
Member

🔗 관련 이슈

📙 작업 내역

  • 홈, 온보딩 에셋 교체 및 자간 수정
  • 알림 센터 UI 수정사항 구현

🎨 스크린샷 또는 시연 영상 (선택)

기능 미리보기 기능 미리보기
온보딩 초입 로그인 화면

@linear
Copy link
Copy Markdown

linear Bot commented May 13, 2026

TWI-84

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 13, 2026

Review Change Stack

Important

Review skipped

Auto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: e3c84e72-bc75-476e-bb26-2972c5db94f4

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
📝 Walkthrough

Walkthrough

이 PR은 알림 목록 항목에 상대 시간을 표시하기 위해 RelativeTimeFormatter를 Date 기반으로 리팩토링하고, 알림 뷰의 레이아웃을 업데이트하며, 이미지 자산을 PNG에서 SVG로 변환하고, 인증 뷰의 제목 텍스트 레이아웃을 정리합니다.

Changes

알림 뷰에 상대 시간 표시

Layer / File(s) Summary
상대 시간 포매터 Date 기반 메서드 추가
Projects/Shared/Util/Sources/RelativeTimeFormatter.swift
RelativeTimeFormatter에 새 public 메서드 displayText(from date: Date) -> String을 추가하여 경과 시간 계산 로직을 Date 처리로 이동하고, 기존 displayText(from raw: String?) 메서드를 ISO8601 파싱 후 새 메서드로 위임하도록 리팩토링합니다.
알림 뷰에 상대 시간 표시 통합
Projects/Feature/Notification/Project.swift, Projects/Feature/Notification/Sources/NotificationView.swift
NotificationView는 SharedUtil을 import하고, 알림 항목 버튼의 메시지 행을 nested HStack으로 재구성하여 메시지 텍스트와 RelativeTimeFormatter를 통한 상대 시간을 함께 표시하며, 알림 기능 모듈의 의존성 설정에 .shared(implements: .util)을 추가합니다.

이미지 자산 및 UI 텍스트 업데이트

Layer / File(s) Summary
이미지 자산 PNG에서 SVG로 변환
Projects/Shared/DesignSystem/Resources/Image/ImageAssets.xcassets/Illustration/illust_invite.imageset/Contents.json, Projects/Shared/DesignSystem/Resources/Image/ImageAssets.xcassets/Illustration/illust_singing.imageset/Contents.json
illust_invite 및 illust_singing 자산 카탈로그 정의에서 PNG 파일 이름을 SVG 파일 이름으로 변경합니다.
인증 뷰 제목 텍스트 단순화
Projects/Feature/Auth/Sources/View/AuthView.swift
AuthView의 titleView를 두 개의 별도 Text 요소에서 Swift 문자열 리터럴로 생성한 단일 multiline Text로 변경하여 동일한 typography 및 foreground 스타일링을 유지하며 코드를 정리합니다.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related PRs

  • Keepiluv/Keepiluv-iOS#123: AuthView.swift의 titleView 텍스트 구성 및 스타일링을 수정하는 동일한 UI 컴포넌트에 대한 변경사항이 겹칩니다.

Suggested labels

🐞 fix

Suggested reviewers

  • jihun32
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/#298/TWI-84

Comment @coderabbitai help to get the list of available commands and usage tips.

@clxxrlove
Copy link
Copy Markdown
Member Author

@coderabbitai summary

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 13, 2026

✅ Actions performed

Summary regeneration triggered.

@clxxrlove clxxrlove changed the title Fix/#298/twi 84 fix: 홈, 설정 QA May 13, 2026
@clxxrlove clxxrlove changed the title fix: 홈, 설정 QA fix: 홈, 설정 QA 반영 May 13, 2026
@clxxrlove clxxrlove added the fix label May 13, 2026
Copy link
Copy Markdown
Contributor

@jihun32 jihun32 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
Contributor

Choose a reason for hiding this comment

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

svg 쓸 때 single scale, prevserve Vector Data 체크 안해주면 깨져보여서 그것들 다 설정해줘
https://medium.com/@Jager-yoo/swiftui-svg-%ED%8F%AC%EB%A7%B7-%EC%9D%B4%EB%AF%B8%EC%A7%80%EB%A1%9C-%EA%B9%A8%EB%81%97%ED%95%9C-ui-%EB%A7%8C%EB%93%A4%EA%B8%B0-7e902a8b5ded

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

처음 안 사실이다... 반영했어!

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

지금보니까 함수에 주석 안달려있는데 하는김에 달아주라..ㅋㅋ

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

완료 ~~

Copy link
Copy Markdown
Contributor

@jihun32 jihun32 left a comment

Choose a reason for hiding this comment

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

리뷰 반영 확인했고 머지할게~

@jihun32 jihun32 merged commit b04df0c into develop May 14, 2026
2 checks passed
@jihun32 jihun32 deleted the fix/#298/TWI-84 branch May 14, 2026 05:54
clxxrlove added a commit that referenced this pull request May 14, 2026
* fix: 변경된 에셋으로 교체 - #298

* feat: 알림 센터에서 시간 시간 표기 추가 - #298

* fix: 로그인 화면 UI 버그 수정 - #298

* fix: 이미지 품질 개선 - #298

* docs: RelativeTimeFormatter에 주석 추가 - #298
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.

2 participants