Skip to content

사이드네비게이션 채팅방 리스트 스크롤 추가#370

Merged
Bangdayeon merged 1 commit intomainfrom
fix/#369-add-sidenavigation-chatroomlist-scroll
Mar 3, 2026
Merged

사이드네비게이션 채팅방 리스트 스크롤 추가#370
Bangdayeon merged 1 commit intomainfrom
fix/#369-add-sidenavigation-chatroomlist-scroll

Conversation

@Bangdayeon
Copy link
Member

@Bangdayeon Bangdayeon commented Mar 1, 2026

관련 이슈

PR 설명

  • 사이드네비게이션의 채팅방 리스트 스크롤 추가

실행 화면

  • Docker에 임의로 테스트 채팅방을 넣어서 체크한 화면입니다.
2026-03-03.5.40.26.mov

@coderabbitai
Copy link

coderabbitai bot commented Mar 1, 2026

검토 내용

사이드 네비게이션 컴포넌트의 레이아웃 스타일링을 조정하는 변경 사항입니다. SideNavigation.tsx에서 래퍼 div에 flex, min-h-0, flex-1, flex-col CSS 클래스를 추가하여 사이드바 콘텐츠의 flex 동작과 높이 처리를 변경했습니다. 동시에 ChatItem.tsx에서 채팅 아이템 컨테이너의 너비 유틸리티를 w-50에서 w-48.5로 변경했습니다. 조건부 렌더링 로직이나 컴포넌트 구조에는 변화가 없습니다.

관련 있을 수 있는 PR

  • ChatRoomSection UI 구현 #329: SideNavigation.tsx와 ChatRoomSection/ChatItem.tsx 동일 파일에 대한 변경으로 컴포넌트 레벨에서 직접적인 관련성을 가집니다.
🚥 Pre-merge checks | ✅ 4
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed PR 제목이 변경사항의 주요 목표인 '사이드네비게이션 채팅방 리스트 스크롤 추가'를 명확하게 설명하고 있습니다.
Linked Issues check ✅ Passed 이슈 #369의 요구사항인 '채팅방 리스트에 스크롤 추가'를 충족하기 위해 SideNavigation 레이아웃 스타일링과 ChatItem 너비 조정이 적용되었습니다.
Out of Scope Changes check ✅ Passed 모든 변경사항이 채팅방 리스트 스크롤 기능 추가라는 PR 목표와 직접 관련된 레이아웃 조정입니다.
Description check ✅ Passed PR 설명이 필요한 섹션을 모두 포함하고 있으며, 관련 이슈 #369를 명시하고 구체적인 변경 사항(채팅방 리스트 스크롤 추가)과 실행 화면을 제공했습니다.

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

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/#369-add-sidenavigation-chatroomlist-scroll

Tip

Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs).
Share your feedback on Discord.


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.

@Bangdayeon Bangdayeon merged commit 72a210c into main Mar 3, 2026
3 checks passed
@Bangdayeon Bangdayeon deleted the fix/#369-add-sidenavigation-chatroomlist-scroll branch March 3, 2026 08:45
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.

[Bug] 사이드바 채팅방리스트 스크롤 추가

2 participants