Skip to content

feat: caption1/medium 추가 #225

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.

Sign up for GitHub

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

Jump to bottom
Merged
doyeonk429 merged 1 commit into from
Oct 18, 2025
Merged

feat: caption1/medium 추가 #225

doyeonk429 merged 1 commit into from
Oct 18, 2025

Conversation

@doyeonk429
Copy link
Member

@doyeonk429 doyeonk429 commented Oct 9, 2025 •
edited by coderabbitai bot
Loading

🔗 관련 이슈

📘 작업 유형

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

📙 작업 내역

  • caption1/medium 추가(기타 프로퍼티 동일)

🧪 테스트 내역

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

✅ PR 체크리스트

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

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

  • 추가로 검토하라 사항 없음. 진짜 폰트만 추가되었습니다.

Summary by CodeRabbit

  • 신기능
    • 캡션1 텍스트 스타일에 Medium 굵기 옵션을 새로 지원합니다. 앱 전반의 캡션 영역에서 Medium 선택 시 적절한 서체, 크기, 행간, 자간이 자동 적용되어 가독성과 시각적 일관성이 향상됩니다. 기존 Regular(기본) 설정에는 변화가 없으며, 디자인 스펙에 맞춘 세밀한 타이포그래피 제어가 가능해졌습니다. 캡션을 많이 사용하는 화면에서 더욱 또렷한 표현을 제공합니다.

@github-actions github-actions bot requested a review from clxxrlove October 9, 2025 11:21
@coderabbitai
Copy link

coderabbitai bot commented Oct 9, 2025 •
edited
Loading

Walkthrough

BKTextStyle.fontAttributes에 caption1 + .medium 가중치 분기를 추가하여 해당 조합의 폰트 이름, 크기, 줄간격, 자간을 반환하도록 했습니다. caption2 분기 앞의 공백 한 줄이 제거되었습니다. 공개 API 시그니처 변경은 없습니다.

Changes

Cohort / File(s) Change Summary
Typography font attributes
src/Projects/BKDesign/Sources/Foundation/Typography/BKTextStyle.swift		 caption1에서 weight == .medium일 때 fontName: .pretendardMedium, fontSize: .pt12, lineHeight: .p133_4, letterSpacing: .pNegative1 반환 분기 추가. caption1의 기존 regular/default 분기는 유지. caption2 이전 불필요 공백 제거.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Pre-merge checks and finishing touches

✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check ✅ Passed PR 제목 ‘feat: caption1/medium 추가’는 캡션1 Medium 스타일 추가라는 주요 변경사항을 정확하고 간결하게 요약하고 있어 변경내용과 일치합니다.
Linked Issues Check ✅ Passed 이 PR에서는 디자인시스템의 폰트 스타일 추가 요구사항(#222)에 따라 BKTextStyle에 caption1/medium 스타일을 추가하여 연관 이슈의 주요 기능을 완전히 구현하였습니다.
Out of Scope Changes Check ✅ Passed 추가된 변경사항은 caption1/medium 스타일 정의와 소소한 빈 줄 제거뿐이며 연관 이슈 외에 불필요하거나 범위를 벗어난 수정이 포함되어 있지 않습니다.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
✨ 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-346-feature/#222

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.

coderabbitai[bot]
coderabbitai bot reviewed Oct 9, 2025
View reviewed changes
Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 0

🧹 Nitpick comments (1)
src/Projects/BKDesign/Sources/Foundation/Typography/BKTextStyle.swift (1)

281-294: 선택적 리팩토링: .regular와 default 케이스 통합 고려

.regular 케이스와 default 케이스가 동일한 값을 반환하고 있습니다. 파일 전체적으로 이런 패턴이 반복되고 있어(예: label2, caption2), 코드 중복을 줄이기 위해 .regular 케이스를 제거하고 default만 사용하는 것을 고려해볼 수 있습니다.

참고: 이 제안은 전체 파일의 일관성을 위한 것이므로, 별도 리팩토링 작업으로 진행하는 것이 적합합니다.

📜 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 a7141ac and 8d10c7e.

📒 Files selected for processing (1)
  • src/Projects/BKDesign/Sources/Foundation/Typography/BKTextStyle.swift (1 hunks)
⏰ 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 (1)
src/Projects/BKDesign/Sources/Foundation/Typography/BKTextStyle.swift (1)

274-280: 변경사항이 잘 구현되었습니다.

caption1에 .medium 가중치가 추가되어 다른 텍스트 스타일들의 .medium 케이스와 일관된 패턴을 따릅니다. 폰트 속성값(.pt12, .p133_4, .pNegative1)도 기존 caption1의 .regular와 동일하게 유지되며, 폰트명만 .pretendardMedium으로 변경되어 올바르게 구현되었습니다.

@doyeonk429 doyeonk429 merged commit 0706a96 into develop Oct 18, 2025
5 checks passed
@doyeonk429 doyeonk429 deleted the BOOK-346-feature/#222 branch October 18, 2025 06:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@coderabbitai coderabbitai[bot] coderabbitai[bot] left review comments

@clxxrlove clxxrlove clxxrlove approved these changes

Assignees

@doyeonk429 doyeonk429

Labels

✨ feat

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[BOOK-346/feat] 디자인시스템 폰트 스타일 추가

3 participants

@doyeonk429 @clxxrlove