[FEAT/#86] 채팅 내역 UI 구현#93
Conversation
|
Thanks for the contribution! Please review the labels and make any necessary changes. |
There was a problem hiding this comment.
Code Review
This pull request implements the chat room list UI for admin and partner pages, including mock data integration and a global update to routing paths. The ChatRoomItem component was enhanced with unread count logic and touch feedback. Review feedback suggests improving the user experience by disabling touch feedback when no action is provided, correcting the empty state description for partner pages to ensure contextual accuracy and proper grammar, and refactoring duplicated list logic into a reusable component to improve maintainability.
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
| className="flex-1 bg-canvas" | ||
| contentContainerClassName="flex-1 px-6 pt-6" | ||
| > | ||
| <View className="px-6"> |
There was a problem hiding this comment.
큰 문제는 아니고 궁금한건데 contentContainerClassName="flex-1 px-6 pt-6" 에서 패딩이 적용되는데 내부 className에 패딩을 또 적용하신 이유가 있을까요!? 패딩이 중복되나 싶어서 여쭤봅니다!
There was a problem hiding this comment.
첨에 작성하고 나중에 더 구현하다가 놓친거같습니다!ㅎㅎ 수정했어용
📝 요약
⚙️ 작업 내용
1.
entities/chat/model/types.tsChatRoomItemProps수정2.
entities/chat/ui/ChatRoomItem.tsx- 채팅방 아이템3.
entities/chat/ui/ChatRoomList.tsx- 채팅방 아이템 리스트FlatList+getItemLayout(length: 70, offset: 70 * index)EmptyState표시4.
pages/admin/chat/model/mockChatRooms.ts, pages/partner/chat/model/mockChatRooms.ts5.
pages/admin/chat/ui/AdminChatPage.tsx, pages/partner/chat/ui/PartnerChatPage.tsx🔗 관련 이슈
✅ 체크리스트
💬 리뷰어에게