Skip to content

[Fix] #161 - typo 적용 - #162

Merged
juri123123 merged 1 commit into
defaultfrom
fix/#161-dialog-qa
Aug 3, 2026
Merged

[Fix] #161 - typo 적용#162
juri123123 merged 1 commit into
defaultfrom
fix/#161-dialog-qa

Conversation

@juri123123

Copy link
Copy Markdown
Contributor

🌴 PR 요약

Dialog 에 typo 적용이 안되고 있던 버그를 수정했습니다.

🌱 작업한 브랜치

🌱 PR Point

attributed를 사용하면서 text를 바꿀 때마다 호출해줘야 해서 해당 사항 반영했습니다.

📌 참고 사항

📸 스크린샷

기능 스크린샷
기능이름 스크린샷 첨부

📮 관련 이슈

@juri123123
juri123123 requested a review from yungu0010 July 29, 2026 13:57
@juri123123 juri123123 self-assigned this Jul 29, 2026
@juri123123 juri123123 added the bug Something isn't working label Jul 29, 2026
@juri123123 juri123123 linked an issue Jul 29, 2026 that may be closed by this pull request
1 task
@coderabbitai

coderabbitai Bot commented Jul 29, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 71f7e534-7f28-445d-ae22-3b90925dd8eb

📥 Commits

Reviewing files that changed from the base of the PR and between 535d7a8 and 82bfb80.

📒 Files selected for processing (1)
  • MDS/Sources/Components/Dialog/MDSDialog.swift

Summary by CodeRabbit

  • 버그 수정
    • 대화상자의 제목과 설명 텍스트에 지정된 글꼴 스타일이 일관되게 적용되도록 개선했습니다.
    • 대화상자 표시 시 제목은 헤딩 스타일, 설명은 본문 스타일로 정상 표시됩니다.

Walkthrough

MDSDialog의 제목 및 설명 라벨 타이포그래피 적용 위치를 프로퍼티 초기화 단계에서 setUI()로 이동했습니다. 텍스트 할당 후 각각 heading3body2가 적용됩니다.

Changes

대화상자 타이포그래피 설정

Layer / File(s) Summary
UI 설정에서 라벨 타이포그래피 적용
MDS/Sources/Components/Dialog/MDSDialog.swift
titleLabeldescriptionLabel 초기화 시 타이포그래피 호출을 제거하고, setUI()에서 텍스트 설정 후 각각 Typography.heading3Typography.body2를 적용합니다.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Suggested reviewers: yungu0010

Poem

당근 들고 토끼가 깡충,
제목엔 heading3 한 송이,
설명엔 body2 포근히,
UI 순서 가지런히,
대화창이 반짝반짝!

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed 제목이 161번 이슈의 Dialog 타이포그래피 수정이라는 핵심 변경을 간결하게 요약합니다.
Description check ✅ Passed 설명이 Dialog에서 타이포그래피가 적용되지 않던 버그 수정과 변경 내용을 잘 설명합니다.
Linked Issues check ✅ Passed 직접 연결된 #161의 Dialog QA 반영 목적과 실제 수정 내용이 일치합니다.
Out of Scope Changes check ✅ Passed 변경은 MDSDialog의 타이포그래피 적용 위치 조정만 포함하며 범위를 벗어난 수정이 없습니다.
✨ Finishing Touches 💡 1
📝 Generate docstrings 💡
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/#161-dialog-qa

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.

backgroundColor = SemanticColor.Bg.Neutral.ghost

titleLabel.text = title
titleLabel.setTypography(Typography.heading3)

Copy link
Copy Markdown
Member

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 Author

Choose a reason for hiding this comment

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

답이 늦엇네요 ㅜㅜ setTypo는 text가 바뀔 때마다 적용해줘야 하는데 이 경우엔 컴포넌트 생성 시점에 text를 정해주는 것이 아니라 text를 정하는 시점에서 폰트를 적용했습니디ㅏ!

@juri123123
juri123123 merged commit 0ca1ed0 into default Aug 3, 2026
1 check passed
@juri123123
juri123123 deleted the fix/#161-dialog-qa branch August 3, 2026 15:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Fix] Dialog QA 반영

2 participants