fix: 모임 검색 컴포넌트 디자인 요구상에 맞춰 수정

[ho0010]

Summary by CodeRabbit

• 스타일
  • 검색 카드 유형의 카드 패딩을 상하 24px, 좌우 12px로 조정하여 하단 여백이 확대되었습니다.
  • 불필요한 주석을 제거해 표시에는 변화 없고 스타일 정의만 정리되었습니다.
• 버그 수정
  • 그룹 검색 결과의 카드에서 ‘진행 중’ 상태가 더 이상 표시되지 않습니다(항상 비표시).

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Preview	Comments	Updated (UTC)
thip	Ready	Preview	Comment	Sep 23, 2025 2:29am

[coderabbitai]

Walkthrough

• GroupCard(search 타입)의 padding을 24px 12px 12px 12px → 24px 12px로 변경하고, 주석을 제거했습니다.
• GroupSearchResult에서 GroupCard에 전달하는 isOngoing 값을 group.isOnGoing 대신 상수 false로 고정했습니다.

Changes

Cohort / File(s)	Summary
GroupCard 스타일 정리 src/components/group/GroupCard.tsx	search 카드 padding을 24px 12px로 변경(하단 패딩 12px→24px). Info의 min-width와 Title width 라인에서 인라인 주석 제거(기능 변화 없음).
검색 결과 렌더링 속성 고정 src/components/search/GroupSearchResult.tsx	GroupCard에 전달하는 isOngoing을 group.isOnGoing에서 false로 고정. 기타 로직 변화 없음.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

• fix: QA 2차 수정 #190: GroupCard.tsx와 GroupSearchResult 사용부 모두를 수정한 PR로, 동일 파일군과 속성/레이아웃 변경이 겹칩니다.
• feat: getSearchRoom API 연동 #142: GroupSearchResult에서 isOngoing 전달 로직을 다룬 변경으로, 이번 PR의 속성 고정과 직접적으로 관련됩니다.
• fix: QA #226: GroupCard.tsx 스타일(edit: cursor 관련) 변경으로, 이번 PR의 스타일 조정과 파일 수준에서 연관됩니다.

Suggested labels

🎨 Html&css, 🐞 BugFix

Suggested reviewers

• heeeeyong
• ljh130334

Poem

작은 카드에 패딩 한 스푼, 폭신하게 깔아두고
진행 중 표시는 살짝 접어, 오늘은 쉼표 하나—false!
토끼는 귀를 쫑긋 세워, UI 선을 다듬어요
좌우는 가볍게, 상하는 넉넉히
픽셀 사이로 폴짝폴짝, 깔끔함에 박수! 🐇✨

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check	✅ Passed	제목은 '모임 검색 컴포넌트'의 디자인 관련 수정을 명확히 요약하고 있어 변경사항과 직접 연관됩니다. PR 요약에 나타난 GroupCard의 패딩 조정 및 GroupSearchResult의 ongoing 상태 처리 변경은 "디자인 요구상에 맞춰 수정"이라는 표현과 일치합니다. 간결하고 팀원이 히스토리를 훑을 때 주요 의도를 빠르게 파악할 수 있습니다.
Docstring Coverage	✅ Passed	No functions found in the changes. Docstring coverage check skipped.

✨ Finishing touches

• 📝 Generate Docstrings

🧪 Generate unit tests

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch fix/QA9-3

Tip

👮 Agentic pre-merge checks are now available in preview!

Pro plan users can now enable pre-merge checks in their settings to enforce checklists before merging PRs.

• Built-in checks – Quickly apply ready-made checks to enforce title conventions, require pull request descriptions that follow templates, validate linked issues for compliance, and more.
• Custom agentic checks – Define your own rules using CodeRabbit’s advanced agentic capabilities to enforce organization-specific policies and workflows. For example, you can instruct CodeRabbit’s agent to verify that API documentation is updated whenever API schema files are modified in a PR. Note: Upto 5 custom checks are currently allowed during the preview period. Pricing for this feature will be announced in a few weeks.

Please see the documentation for more information.

Example:

reviews:
  pre_merge_checks:
    custom_checks:
      - name: "Undocumented Breaking Changes"
        mode: "warning"
        instructions: |
          Pass/fail criteria: All breaking changes to public APIs, CLI flags, environment variables, configuration keys, database schemas, or HTTP/GraphQL endpoints must be documented in the "Breaking Change" section of the PR description and in CHANGELOG.md. Exclude purely internal or private changes (e.g., code not exported from package entry points or explicitly marked as internal).

Please share your feedback with us on this Discord post.

---

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

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Actionable comments posted: 0

🧹 Nitpick comments (3)

src/components/group/GroupCard.tsx (1)

67-67: border-radius에 'none'은 유효한 값이 아닙니다.
의도대로 모서리를 0으로 하려면 0을 사용하세요.

적용 제안:

-  border-radius: ${({ cardType }) => (cardType === 'search' ? `none` : '12px')};
+  border-radius: ${({ cardType }) => (cardType === 'search' ? '0' : '12px')};

src/components/search/GroupSearchResult.tsx (2)

103-103: isOngoing을 false로 고정 — 기획 확인 필요.
검색 카드에서 항상 “모집 마감” 라벨을 노출하려는 의도라면 OK입니다. 다만 기존 매핑(isOnGoing ← r.isPublic)은 의미가 달라 혼동 여지가 있습니다. 차후에는 GroupCard가 cardType === 'search'일 때 내부적으로 라벨을 결정하도록 하거나, 속성명을 isRecruiting 등으로 바꿔 의미 충돌을 제거하는 걸 권장합니다.

---

143-150: 스크롤바 숨김 처리 도입 — 접근성/플랫폼 확인 권장.
시각적 요구사항 충족은 되지만, Windows 환경에서 스크롤 존재 인지가 떨어질 수 있습니다. 키보드 스크롤(↑/↓, PgUp/PgDn) 및 마우스 휠 동작, 포커스 트랩 이슈 여부를 한번 점검 부탁.

📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 50ec1d7 and 6f54ab0.

📒 Files selected for processing (2)

• src/components/group/GroupCard.tsx (3 hunks)
• src/components/search/GroupSearchResult.tsx (2 hunks)

🔇 Additional comments (3)

src/components/group/GroupCard.tsx (3)

69-69: 검색 카드 패딩 변경 OK (하단 24px로 증가).
디자인 요구 반영으로 보이며 레이아웃 이탈 없으면 그대로 가도 됩니다. 검색 리스트 컨테이너(검색 결과 영역)와의 간격/스크롤 여백만 실제 화면에서 한번 확인 부탁.

---

114-114: min-width: 0 추가 적절.
플렉스 컨테이너 내 텍스트 생략(ellipsis) 이슈 방지에 필요합니다.

---

128-128: Title 폭 100% 지정 적절.
텍스트 오버플로 제어에 도움이 됩니다.