Skip to content

Conversation

@doyeonk429
Copy link
Member

@doyeonk429 doyeonk429 commented Nov 5, 2025

🔗 관련 이슈

📘 작업 유형

  • ✨ Feature (기능 추가)
  • 🐞 Bugfix (버그 수정)
  • 🔧 Refactor (코드 리팩토링)
  • ⚙️ Chore (환경 설정)
  • 📝 Docs (문서 작성 및 수정)
  • ✅ Test (기능 테스트)
  • 🎨 style (코드 스타일 수정)

📙 작업 내역

  • 이탤릭체 적용
  • 문장 부호 추가
  • 이미지 에셋 추가

🧪 테스트 내역

  • 브라우저/기기에서 동작 확인
  • 엣지 케이스 테스트 완료
  • 기존 기능 영향 없음

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

TO-BE

Simulator Screenshot - ReedDefaultSize - 2025-11-05 at 18 52 13 Simulator Screenshot - ReedDefaultSize - 2025-11-05 at 18 52 10

✅ PR 체크리스트

  • 커밋 메시지가 명확합니다
  • PR 제목이 컨벤션에 맞습니다
  • 관련 이슈 번호를 작성했습니다
  • 기능이 정상적으로 작동합니다
  • 불필요한 코드를 제거했습니다

💬 추가 설명 or 리뷰 포인트 (선택)

  • 기존에 이탤릭체 만들어 두셨는데 적용이 안되었던 것 같아서 해당 부분 적용했습니다!

Summary by CodeRabbit

새로운 기능

  • 감정 표현 원형 그래픽 네 가지 추가 (통찰, 기쁨, 슬픔, 따뜻함)
  • 앱 전체에서 향상된 감정 아이콘 시스템 적용

개선사항

  • 수집된 문장 표시 시 따옴표 자동 추가
  • 페이지 라벨 텍스트 스타일 최적화

@doyeonk429 doyeonk429 self-assigned this Nov 5, 2025
@doyeonk429 doyeonk429 added the 🐞 fix Something isn't working label Nov 5, 2025
@github-actions github-actions bot requested a review from clxxrlove November 5, 2025 10:03
@coderabbitai
Copy link

coderabbitai bot commented Nov 5, 2025

Walkthrough

네 개의 새로운 감정 원형 이미지 에셋을 Xcode 에셋 카탈로그에 추가하고, BKImage API에 해당 접근자를 추가하며, UI 컴포넌트를 업데이트하여 새로운 에셋을 참조하고 폰트 스타일 및 텍스트 서식을 변경합니다.

Changes

Cohort / File(s) Summary
Asset Catalog 에셋 정의
src/Projects/BKDesign/Resources/Assets.xcassets/graphics/insight_circle.imageset/Contents.json, src/Projects/BKDesign/Resources/Assets.xcassets/graphics/joy_circle.imageset/Contents.json, src/Projects/BKDesign/Resources/Assets.xcassets/graphics/sad_circle.imageset/Contents.json, src/Projects/BKDesign/Resources/Assets.xcassets/graphics/warm_circle.imageset/Contents.json
각 감정별 원형 이미지 에셋(1x, 2x, 3x 해상도)에 대한 Contents.json 파일 추가
GraphicSystem API 확장
src/Projects/BKDesign/Sources/Foundation/GraphicSystem/BKImage.swift
BKImage.Graphics에 warmCircle, joyCircle, sadCircle, insightCircle 공개 정적 프로퍼티 추가
UI 컴포넌트 업데이트
src/Projects/BKPresentation/Sources/MainFlow/NoteCompletion/View/AppreciationResultView.swift
EmotionIcon에서 기존 에셋(warm, joy, sad, insight)을 새로운 원형 에셋으로 변경
UI 스타일 및 서식 개선
src/Projects/BKPresentation/Sources/MainFlow/NoteCompletion/View/CollectedSentenceView.swift
pageLabel 폰트 스타일을 italic으로 변경 및 수집한 문장 양쪽에 따옴표(") 추가

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

  • 주의할 점:
    • AppreciationResultView.swift의 에셋 변경사항이 의도한 UI 디자인과 일치하는지 확인
    • CollectedSentenceView.swift의 폰트 스타일 변경이 디자인 스펙(SF Pro Medium Italic 14pt)과 일치하는지 확인
    • 따옴표 추가로 인한 텍스트 레이아웃 변경사항 검증

Pre-merge checks and finishing touches

✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed Pull request title accurately summarizes the main changes: font and image fixes in the note details screen, which matches the modifications to CollectedSentenceView (font styling) and AppreciationResultView (image assets).
Linked Issues check ✅ Passed Pull request addresses all linked issue #245 objectives: quotation marks added to sentences, italic font applied, new circle image assets created and integrated, though image size/styling (40px, background color, radius) implementation details are not fully evident in code changes.
Out of Scope Changes check ✅ Passed All changes are directly related to the linked issue requirements: new asset files for circle images, BKImage accessors for the graphics, and UI updates in CollectedSentenceView and AppreciationResultView align with the specified objectives.
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch BOOK-410-fix/#245

📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between de0cff9 and 41a0ade.

⛔ Files ignored due to path filters (12)
  • src/Projects/BKDesign/Resources/Assets.xcassets/graphics/insight_circle.imageset/insight_circle.png is excluded by !**/*.png
  • src/Projects/BKDesign/Resources/Assets.xcassets/graphics/insight_circle.imageset/insight_circle@2x.png is excluded by !**/*.png
  • src/Projects/BKDesign/Resources/Assets.xcassets/graphics/insight_circle.imageset/insight_circle@3x.png is excluded by !**/*.png
  • src/Projects/BKDesign/Resources/Assets.xcassets/graphics/joy_circle.imageset/joy_circle.png is excluded by !**/*.png
  • src/Projects/BKDesign/Resources/Assets.xcassets/graphics/joy_circle.imageset/joy_circle@2x.png is excluded by !**/*.png
  • src/Projects/BKDesign/Resources/Assets.xcassets/graphics/joy_circle.imageset/joy_circle@3x.png is excluded by !**/*.png
  • src/Projects/BKDesign/Resources/Assets.xcassets/graphics/sad_circle.imageset/sad_circle.png is excluded by !**/*.png
  • src/Projects/BKDesign/Resources/Assets.xcassets/graphics/sad_circle.imageset/sad_circle@2x.png is excluded by !**/*.png
  • src/Projects/BKDesign/Resources/Assets.xcassets/graphics/sad_circle.imageset/sad_circle@3x.png is excluded by !**/*.png
  • src/Projects/BKDesign/Resources/Assets.xcassets/graphics/warm_circle.imageset/warm_circle.png is excluded by !**/*.png
  • src/Projects/BKDesign/Resources/Assets.xcassets/graphics/warm_circle.imageset/warm_circle@2x.png is excluded by !**/*.png
  • src/Projects/BKDesign/Resources/Assets.xcassets/graphics/warm_circle.imageset/warm_circle@3x.png is excluded by !**/*.png
📒 Files selected for processing (7)
  • src/Projects/BKDesign/Resources/Assets.xcassets/graphics/insight_circle.imageset/Contents.json (1 hunks)
  • src/Projects/BKDesign/Resources/Assets.xcassets/graphics/joy_circle.imageset/Contents.json (1 hunks)
  • src/Projects/BKDesign/Resources/Assets.xcassets/graphics/sad_circle.imageset/Contents.json (1 hunks)
  • src/Projects/BKDesign/Resources/Assets.xcassets/graphics/warm_circle.imageset/Contents.json (1 hunks)
  • src/Projects/BKDesign/Sources/Foundation/GraphicSystem/BKImage.swift (1 hunks)
  • src/Projects/BKPresentation/Sources/MainFlow/NoteCompletion/View/AppreciationResultView.swift (1 hunks)
  • src/Projects/BKPresentation/Sources/MainFlow/NoteCompletion/View/CollectedSentenceView.swift (2 hunks)
🧰 Additional context used
🧠 Learnings (3)
📓 Common learnings
Learnt from: doyeonk429
Repo: YAPP-Github/Reed-iOS PR: 157
File: src/Projects/BKPresentation/Sources/MainFlow/Home/View/HomeViewController.swift:19-24
Timestamp: 2025-08-08T01:38:59.656Z
Learning: doyeonk429는 Reed-iOS 프로젝트에서 접근성(accessibility) 관련 개선사항은 현재 작업 중인 PR에서 즉시 처리하지 않고, 접근성 전용 PR이나 이슈를 별도로 만들어 한번에 처리하는 것을 선호한다.
Learnt from: doyeonk429
Repo: YAPP-Github/Reed-iOS PR: 157
File: src/Projects/BKPresentation/Sources/AuthFlow/View/LoginView.swift:43-45
Timestamp: 2025-08-08T01:39:15.620Z
Learning: doyeonk429는 Reed-iOS 프로젝트에서 접근성 개선 작업을 별도의 전용 PR이나 이슈에서 일괄 처리하는 것을 선호한다. 개별 기능 구현 PR에서는 접근성 관련 제안을 하지 않고, 접근성 전담 작업에서 한번에 처리하는 방식을 원한다.
Learnt from: doyeonk429
Repo: YAPP-Github/Reed-iOS PR: 163
File: src/Projects/BKData/Sources/DTO/Request/UserBookRegisterRequestDTO.swift:7-9
Timestamp: 2025-08-08T16:13:31.712Z
Learning: Reed-iOS에서 서버 스펙 변경으로 DTO 필드명이 바뀔 때, doyeonk429는 코드 전반의 파라미터/시그니처 리네이밍까지 확장하지 않고, 실제 요청 직렬화 키가 맞는지(예: isbn13)만 보장하면 충분하다는 범위 최소화 방식을 선호한다. (PR: YAPP-Github/Reed-iOS#163, 파일: BKData/Sources/DTO/Request/UserBookRegisterRequestDTO.swift)
Learnt from: clxxrlove
Repo: YAPP-Github/Reed-iOS PR: 112
File: src/Projects/BKPresentation/Sources/MainFlow/Note/View/EmotionRegistrationView.swift:52-53
Timestamp: 2025-07-28T10:44:32.256Z
Learning: clxxrlove는 EmotionRegistrationView에서 someEmotion1-4 케이스들을 임시로 구현했으며, 해당하는 그래픽 디자인이 나온 후 수정할 예정이라고 명시했다. 감정 선택 UI는 디자인 에셋 완성 후 구현하는 것을 선호한다.
Learnt from: clxxrlove
Repo: YAPP-Github/Reed-iOS PR: 69
File: src/Projects/BKDesign/Resources/Assets.xcassets/AppIcon.appiconset/Contents.json:1-35
Timestamp: 2025-07-10T13:26:07.554Z
Learning: clxxrlove는 Reed-iOS 프로젝트에서 AppIcon 에셋 구조를 미리 설정하고, 실제 이미지 파일이 준비되기 전까지는 filename 속성을 의도적으로 생략하는 개발 방식을 사용한다. 이는 에셋 구조를 먼저 만들고 나중에 이미지를 추가하는 단계적 개발 접근법이다.
Learnt from: doyeonk429
Repo: YAPP-Github/Reed-iOS PR: 68
File: src/Projects/BKDesign/PreviewApp/Sources/View/BKButtonTestViewController.swift:124-133
Timestamp: 2025-07-10T08:21:49.399Z
Learning: doyeonk429는 테스트 전용으로만 사용되는 extension이나 코드는 해당 테스트 파일에 그대로 두는 것을 선호합니다. 실제 프로덕션 코드에서 사용되지 않는 테스트 전용 코드는 별도 파일로 분리하지 않고 테스트 파일 내에 유지하는 것이 그들의 코드 구성 방식입니다.
Learnt from: doyeonk429
Repo: YAPP-Github/Reed-iOS PR: 138
File: src/Projects/BKDesign/Resources/Assets.xcassets/icons/home.imageset/Contents.json:4-14
Timestamp: 2025-08-04T15:20:43.982Z
Learning: doyeonk429는 Reed-iOS 프로젝트에서 에셋 파일명이 영어가 아닌 경우(한글, 일본어 등)에는 경고를 받고 싶어하지만, 영어 파일명의 네이밍 컨벤션(예: home.png vs home1.png)은 기능적으로 문제없다면 신경쓰지 않는 것을 선호한다.
Learnt from: doyeonk429
Repo: YAPP-Github/Reed-iOS PR: 98
File: src/Projects/BKPresentation/Sources/AuthFlow/View/TermsView.swift:78-85
Timestamp: 2025-07-22T05:37:28.756Z
Learning: doyeonk429는 Reed-iOS 프로젝트에서 더미 데이터나 테스트 데이터의 구체적인 내용(예: URL 주소)에 대해서는 리뷰 시 지적하지 않기를 선호한다. 더미 데이터는 임시적이며 실제 기능 구현 시 교체될 예정이므로 URL의 구체적인 주소는 중요하지 않다고 본다.
📚 Learning: 2025-07-08T17:15:59.793Z
Learnt from: doyeonk429
Repo: YAPP-Github/Reed-iOS PR: 63
File: src/Projects/BKDesign/Resources/Assets.xcassets/chevron-right.imageset/Contents.json:1-26
Timestamp: 2025-07-08T17:15:59.793Z
Learning: In the BKDesign project, all image assets referenced in Contents.json files are properly included in the repository with the correct 1x, 2x, and 3x variants. The chevron-right icon and other design system icons are complete and ready for use.

Applied to files:

  • src/Projects/BKPresentation/Sources/MainFlow/NoteCompletion/View/AppreciationResultView.swift
  • src/Projects/BKDesign/Resources/Assets.xcassets/graphics/joy_circle.imageset/Contents.json
  • src/Projects/BKDesign/Resources/Assets.xcassets/graphics/sad_circle.imageset/Contents.json
  • src/Projects/BKDesign/Sources/Foundation/GraphicSystem/BKImage.swift
  • src/Projects/BKDesign/Resources/Assets.xcassets/graphics/warm_circle.imageset/Contents.json
  • src/Projects/BKDesign/Resources/Assets.xcassets/graphics/insight_circle.imageset/Contents.json
📚 Learning: 2025-07-28T10:44:32.256Z
Learnt from: clxxrlove
Repo: YAPP-Github/Reed-iOS PR: 112
File: src/Projects/BKPresentation/Sources/MainFlow/Note/View/EmotionRegistrationView.swift:52-53
Timestamp: 2025-07-28T10:44:32.256Z
Learning: clxxrlove는 EmotionRegistrationView에서 someEmotion1-4 케이스들을 임시로 구현했으며, 해당하는 그래픽 디자인이 나온 후 수정할 예정이라고 명시했다. 감정 선택 UI는 디자인 에셋 완성 후 구현하는 것을 선호한다.

Applied to files:

  • src/Projects/BKPresentation/Sources/MainFlow/NoteCompletion/View/AppreciationResultView.swift
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
  • GitHub Check: build
🔇 Additional comments (8)
src/Projects/BKPresentation/Sources/MainFlow/NoteCompletion/View/CollectedSentenceView.swift (2)

29-33: LGTM! 이탤릭 폰트가 올바르게 적용되었습니다.

이슈 요구사항에 따라 페이지 레이블에 SF Pro Medium Italic 14pt 폰트가 적용되었습니다.


77-84: LGTM! 따옴표가 올바르게 추가되었습니다.

이슈 요구사항에 따라 수집한 문장 주위에 따옴표가 적절히 추가되었습니다.

src/Projects/BKDesign/Resources/Assets.xcassets/graphics/insight_circle.imageset/Contents.json (1)

1-23: LGTM! 에셋 카탈로그가 올바르게 정의되었습니다.

1x, 2x, 3x 변형이 표준 구조에 따라 적절히 정의되었습니다.

src/Projects/BKDesign/Resources/Assets.xcassets/graphics/warm_circle.imageset/Contents.json (1)

1-23: LGTM! 에셋 카탈로그가 올바르게 정의되었습니다.

1x, 2x, 3x 변형이 표준 구조에 따라 적절히 정의되었습니다.

src/Projects/BKDesign/Resources/Assets.xcassets/graphics/sad_circle.imageset/Contents.json (1)

1-23: LGTM! 에셋 카탈로그가 올바르게 정의되었습니다.

1x, 2x, 3x 변형이 표준 구조에 따라 적절히 정의되었습니다.

src/Projects/BKDesign/Resources/Assets.xcassets/graphics/joy_circle.imageset/Contents.json (1)

1-23: LGTM! 에셋 카탈로그가 올바르게 정의되었습니다.

1x, 2x, 3x 변형이 표준 구조에 따라 적절히 정의되었습니다.

src/Projects/BKDesign/Sources/Foundation/GraphicSystem/BKImage.swift (1)

81-84: LGTM! Public API가 올바르게 확장되었습니다.

새로운 원형 이미지 에셋에 대한 접근자가 기존 패턴과 일관되게 추가되었습니다.

src/Projects/BKPresentation/Sources/MainFlow/NoteCompletion/View/AppreciationResultView.swift (1)

14-21: LGTM! 새로운 원형 이미지 에셋이 올바르게 적용되었습니다.

이슈 요구사항에 따라 원형 이미지 에셋으로 업데이트되었으며, 이미지 크기도 40px로 올바르게 설정되었습니다 (Line 166).


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@doyeonk429 doyeonk429 merged commit 9216662 into develop Nov 7, 2025
5 checks passed
@doyeonk429 doyeonk429 deleted the BOOK-410-fix/#245 branch November 7, 2025 23:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🐞 fix Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BOOK-410/qa] 기록 상세 화면에서 폰트, 이미지 오류

3 participants