Conversation
Walkthrough신고 제출 흐름에 로딩 및 중복 제출 방지 로직을 추가하고, 실패 시 3초 자동 닫힘하는 별도 에러 모달을 새로 등록하여 에러 표시 방식을 변경했습니다. Changes
Sequence Diagram(s)mermaid Estimated code review effort🎯 2 (Simple) | ⏱️ ~8 minutes Possibly related PRs
Suggested reviewers
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches
🧪 Generate unit tests (beta)
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. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Fix all issues with AI agents
In `@src/components/modal/ReportErrorModal.tsx`:
- Around line 30-31: In ReportErrorModal, the paragraph under the
already-reported heading contains a placeholder ("어쩌구저쩌구"); update the
user-facing copy in the <p> element (inside ReportErrorModal, the element with
classes "body-10 desktop:body-6 text-gray-700") to a proper Korean guidance
message (e.g., explain that the keyword has already been reported, what to
expect next, and how to contact support if needed) so the modal shows a real
instructional message instead of the placeholder.
| <h3 className="body-7 desktop:title-3 text-gray-800">이미 신고된 키워드입니다.</h3> | ||
| <p className="body-10 desktop:body-6 text-gray-700">어쩌구저쩌구</p> |
There was a problem hiding this comment.
사용자 메시지에 플레이스홀더 문구가 남아 있습니다.
실제 안내 문구로 교체해주세요.
📝 제안 수정안
- <p className="body-10 desktop:body-6 text-gray-700">어쩌구저쩌구</p>
+ <p className="body-10 desktop:body-6 text-gray-700">
+ 이미 신고된 키워드입니다. 다른 키워드를 선택해 주세요.
+ </p>📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| <h3 className="body-7 desktop:title-3 text-gray-800">이미 신고된 키워드입니다.</h3> | |
| <p className="body-10 desktop:body-6 text-gray-700">어쩌구저쩌구</p> | |
| <h3 className="body-7 desktop:title-3 text-gray-800">이미 신고된 키워드입니다.</h3> | |
| <p className="body-10 desktop:body-6 text-gray-700"> | |
| 이미 신고된 키워드입니다. 다른 키워드를 선택해 주세요. | |
| </p> |
🤖 Prompt for AI Agents
In `@src/components/modal/ReportErrorModal.tsx` around lines 30 - 31, In
ReportErrorModal, the paragraph under the already-reported heading contains a
placeholder ("어쩌구저쩌구"); update the user-facing copy in the <p> element (inside
ReportErrorModal, the element with classes "body-10 desktop:body-6
text-gray-700") to a proper Korean guidance message (e.g., explain that the
keyword has already been reported, what to expect next, and how to contact
support if needed) so the modal shows a real instructional message instead of
the placeholder.
There was a problem hiding this comment.
🧹 Nitpick comments (1)
src/app/(main)/teampsylog/_components/BottomComment.tsx (1)
41-42:maxHeight중복 및h-full레이아웃 오버플로우두 가지 연관된 문제가 있습니다.
maxHeight중복:height가 이미'80vh'로 고정되어 있으므로maxHeight: '80vh'는 아무런 효과가 없습니다.maxHeight는height가auto이거나 가변적일 때만 의미가 있습니다. 의도가 "컨텐츠가 적으면 더 작게, 최대 80vh"였다면height: 'auto',maxHeight: '80vh'로 설정해야 합니다. 단순히 고정 높이라면maxHeight를 제거하면 됩니다.
h-full오버플로우: 62번 줄의 컨텐츠 div에h-full이 적용되면 부모의 높이인 80vh를 그대로 가져갑니다. 그런데 부모 내부에는 드래그 핸들 div(pt-3 pb-2 + 3px 바 ≈ 약 36~40px)도 포함되어 있으므로, 실제 총 높이가 80vh를 초과하여 바텀 시트 컨테이너를 벗어납니다. 부모에overflow: hidden이 없기 때문에 스크롤 영역이 라운드 컨테이너 밖으로 누출될 수 있습니다.Flexbox 레이아웃으로 개선하면 드래그 핸들과 스크롤 영역 간의 높이 분배를 정확하게 처리할 수 있습니다.
♻️ Flexbox 레이아웃 + maxHeight 정리 제안
<div className="desktop:hidden fixed inset-x-0 bottom-0 z-50 rounded-t-2xl bg-gray-200" style={{ - height: '80vh', maxHeight: '80vh', transform: isDragging ? `translateY(${dragCurrentY}px)` : isOpen && !isClosing ? 'translateY(0)' : 'translateY(100%)', transition: isDragging ? 'none' : 'transform 0.3s ease-out', visibility: shouldShow ? 'visible' : 'hidden', pointerEvents: shouldShow ? 'auto' : 'none', }} + className="desktop:hidden fixed inset-x-0 bottom-0 z-50 flex flex-col rounded-t-2xl bg-gray-200" > <div className="flex cursor-grab justify-center pt-3 pb-2 active:cursor-grabbing" ... > <div className="h-[3px] w-9.5 rounded-full bg-gray-400" /> </div> - <div className="h-full overflow-y-auto pb-5">{children}</div> + <div className="min-h-0 flex-1 overflow-y-auto pb-5">{children}</div> </div>
height: auto+maxHeight: '80vh'조합을 사용하면 컨텐츠 적을 때 자연스럽게 줄어들고, flexbox의flex-1 min-h-0으로 스크롤 영역이 나머지 공간을 정확히 채웁니다. (min-h-0은 flex item이overflow: auto와 함께 올바르게 축소되도록 하는 데 필요합니다.)Also applies to: 62-62
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@src/app/`(main)/teampsylog/_components/BottomComment.tsx around lines 41 - 42, The container in BottomComment.tsx sets both height: '80vh' and maxHeight: '80vh' and a child content div uses h-full causing overflow; change the container to height: 'auto' and maxHeight: '80vh' (or simply remove maxHeight if you want a fixed 80vh), make the container a column flexbox, add overflow-hidden on the container, and replace the child content div's h-full with flex-1 min-h-0 and overflow-auto so the drag handle (the pt-3 pb-2 div) no longer causes the scroll area to overflow the rounded bottom sheet.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Nitpick comments:
In `@src/app/`(main)/teampsylog/_components/BottomComment.tsx:
- Around line 41-42: The container in BottomComment.tsx sets both height: '80vh'
and maxHeight: '80vh' and a child content div uses h-full causing overflow;
change the container to height: 'auto' and maxHeight: '80vh' (or simply remove
maxHeight if you want a fixed 80vh), make the container a column flexbox, add
overflow-hidden on the container, and replace the child content div's h-full
with flex-1 min-h-0 and overflow-auto so the drag handle (the pt-3 pb-2 div) no
longer causes the scroll area to overflow the rounded bottom sheet.
1 participant
✅ PR 유형
어떤 변경 사항이 있었나요?
📌 관련 이슈번호
✅ Key Changes
📸 스크린샷 or 실행영상
2026-02-03.000155.mp4
🎸 기타 사항 or 추가 코멘트
Summary by CodeRabbit
새로운 기능
버그 수정