Skip to content

fix: 관심학교 카드 너비 보정#547

Merged
manNomi merged 1 commit into
mainfrom
fix/546-favorite-card-full-width
Jun 3, 2026
Merged

fix: 관심학교 카드 너비 보정#547
manNomi merged 1 commit into
mainfrom
fix/546-favorite-card-full-width

Conversation

@manNomi
Copy link
Copy Markdown
Contributor

@manNomi manNomi commented Jun 3, 2026

관련 이슈

작업 내용

  • 관심학교 목록에서 대학 카드가 콘텐츠 너비만큼만 렌더링되는 문제를 수정했습니다.
  • 공통 UniversityCard의 최상위 Link가 부모 컨테이너 너비를 채우도록 w-full, flex-1, min-w-0을 적용했습니다.
  • 카드 내부 영역은 기존처럼 w-full을 유지하되, flex row 내부에서도 동일한 폭으로 표시되도록 보정했습니다.

특이 사항

  • 로컬 검증 중 pnpm --filter @solid-connect/web run lint:check, pnpm --filter @solid-connect/web run typecheck:ci는 통과했습니다.
  • 로컬 pre-push 훅의 next build는 전체 정적 페이지 생성 단계에서 Unsupported Server Component type 에러로 실패했습니다. 변경 파일은 카드 스타일 1개뿐이고, 로컬 Node가 프로젝트 요구 버전(22.x)과 다른 23.10.0인 상태에서 발생한 전체 빌드 오류라 본 수정과는 별개로 보입니다.
  • 위 빌드 실패 과정에서 Next가 tsconfig.json을 자동 변경했으나, 해당 변경은 작업 범위가 아니므로 반영하지 않았습니다.

리뷰 요구사항 (선택)

  • 관심학교 편집 모드에서 체크 버튼이 노출될 때도 카드가 남은 영역을 자연스럽게 채우는지 확인 부탁드립니다.

@manNomi manNomi requested review from enunsnv and wibaek as code owners June 3, 2026 06:07
@vercel
Copy link
Copy Markdown

vercel Bot commented Jun 3, 2026

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

Project Deployment Actions Updated (UTC)
solid-connect-university-web Error Error Jun 3, 2026 6:08am
solid-connection-web Ready Ready Preview, Comment Jun 3, 2026 6:08am
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
solid-connect-web-admin Skipped Skipped Jun 3, 2026 6:08am

@github-actions github-actions Bot added the web label Jun 3, 2026
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Jun 3, 2026

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

Run ID: c132a563-1474-47e5-9d1f-baa411f4f05f

📥 Commits

Reviewing files that changed from the base of the PR and between d0779ed and 0b2f063.

📒 Files selected for processing (1)
  • apps/web/src/components/ui/UniverSityCard/index.tsx

Walkthrough

UniversityCard 컴포넌트의 Link 요소가 다음과 같이 개선되었습니다:

  1. className 확장: "block"에서 "block w-full min-w-0 flex-1"로 업데이트되어 전체 너비(w-full)를 차지하고 최소 너비 제약(min-w-0)을 적용한 후 유연한 flex 비율(flex-1)을 지원합니다.

  2. 속성 재배치: 단일 라인 구조에서 다중 라인 구조로 변경되어 hrefaria-labelledby 속성의 가독성이 개선되었습니다.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Suggested reviewers

  • wibaek
  • enunsnv
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed PR 제목이 변경의 핵심을 명확하게 설명합니다. '관심학교 카드 너비 보정'은 실제 코드 변경(UniversityCard Link의 너비 관련 클래스 추가)과 정확히 일치합니다.
Description check ✅ Passed PR 설명이 템플릿 구조를 따르며 관련 이슈, 작업 내용, 특이 사항, 리뷰 요구사항을 모두 포함하고 있습니다. 변경 의도와 로컬 검증 결과가 구체적으로 기술되어 있습니다.
Linked Issues check ✅ Passed 코드 변경이 #546에서 요구한 '관심학교 카드 너비 문제 해결'의 구현 요구사항을 완벽하게 충족합니다. UniversityCard의 최상위 Link에 w-full, flex-1, min-w-0을 적용하여 부모 컨테이너 너비를 채우도록 수정했습니다.
Out of Scope Changes check ✅ Passed 모든 변경이 #546 해결 범위 내에 있습니다. UniversityCard 컴포넌트의 Link 요소 스타일 보정만 수행되었으며, 자동 생성된 tsconfig.json 변경은 반영하지 않아 범위를 유지했습니다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/546-favorite-card-full-width

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.

@manNomi manNomi merged commit 3764f34 into main Jun 3, 2026
10 of 11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

관심학교 대학 카드가 전체 너비를 채우지 못하는 UI 깨짐

1 participant