Skip to content

채팅 타이포 시스템 조정#448

Merged
Seong-Myeong merged 1 commit intomainfrom
feature/#447-edit-chatting-type-system
Mar 26, 2026
Merged

채팅 타이포 시스템 조정#448
Seong-Myeong merged 1 commit intomainfrom
feature/#447-edit-chatting-type-system

Conversation

@Seong-Myeong
Copy link
Copy Markdown
Contributor

관련 이슈

PR 설명

  • .ChatPage.tsx 채팅 답변 글꼴에 대한 타이포 조정하였습니다.
image

@Seong-Myeong Seong-Myeong linked an issue Mar 25, 2026 that may be closed by this pull request
@Seong-Myeong Seong-Myeong self-assigned this Mar 25, 2026
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Mar 25, 2026

주요 변경 사항

이번 PR는 ChatPage.tsx 파일의 메시지 관련 스타일링을 수정합니다. 메시지 컨테이너에서 text-sm Tailwind 클래스를 제거하고, AI 답변 탭의 콘텐츠 래퍼에서 text-smfont-body-md로 변경했습니다. 기존의 조건부 정렬 로직(justify-end/justify-start)과 상단 여백은 유지됩니다. 총 2줄이 추가되고 2줄이 제거되었습니다.

관련된 PR들

🚥 Pre-merge checks | ✅ 4
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed The PR title '채팅 타이포 시스템 조정' directly describes the main change: adjusting typography system for chat replies.
Description check ✅ Passed The PR description follows the required template with linked issue (#447) and clear explanation of changes made to ChatPage.tsx typography.
Linked Issues check ✅ Passed The code changes align with issue #447 objective to adjust chat reply typography - removed text-sm and replaced with font-body-md class [#447].
Out of Scope Changes check ✅ Passed All changes in ChatPage.tsx are scoped to typography adjustments for chat messages and AI response styling, directly addressing issue #447 requirements.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feature/#447-edit-chatting-type-system

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.

Copy link
Copy Markdown

@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: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Inline comments:
In `@src/app/`(route)/chat/[id]/ChatPage.tsx:
- Around line 451-453: The change to the outer container's typography is leaking
into user bubbles; update the ChatPage message rendering so that only
assistant/response bubbles get the adjusted typography while user bubbles
explicitly retain the original size—locate the message render in ChatPage.tsx
(the block using message.role and index) and add/restore an explicit text-size
class (e.g., the original "text-sm") on the element that renders user messages
(when message.role === 'user') or alternatively apply the adjusted text class
only when message.role !== 'user', ensuring message, index and the containing
JSX for the bubble are the referenced symbols to modify.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: 8e01b83d-5866-4bb2-8aee-b807492bd845

📥 Commits

Reviewing files that changed from the base of the PR and between 488e088 and a4e68b2.

📒 Files selected for processing (1)
  • src/app/(route)/chat/[id]/ChatPage.tsx

@Seong-Myeong Seong-Myeong merged commit 69b8de2 into main Mar 26, 2026
3 checks passed
@Seong-Myeong Seong-Myeong deleted the feature/#447-edit-chatting-type-system branch March 26, 2026 11:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

채팅 답변 타이포 시스템 조정

2 participants