Skip to content

fix: 대학 상세 정보 페이지 좌우 패딩 제거#465

Open
manNomi wants to merge 1 commit intomainfrom
fix/university-detail-remove-px5
Open

fix: 대학 상세 정보 페이지 좌우 패딩 제거#465
manNomi wants to merge 1 commit intomainfrom
fix/university-detail-remove-px5

Conversation

@manNomi
Copy link
Contributor

@manNomi manNomi commented Mar 2, 2026

Summary

  • 대학 상세 정보 페이지 래퍼 클래스에서 px-5를 제거해 가로 여백을 없앴습니다.
  • w-full은 유지해 전체 너비 레이아웃은 그대로 동작하도록 했습니다.

Verification

  • lsp_diagnostics (apps/web/src/app/university/[homeUniversity]/[id]/page.tsx) 에러 없음
  • pnpm --filter @solid-connect/web run typecheck 통과
  • pnpm --filter @solid-connect/web run build 통과

@manNomi manNomi requested review from enunsnv, khwww and wibaek as code owners March 2, 2026 14:04
@vercel
Copy link

vercel bot commented Mar 2, 2026

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

Project Deployment Actions Updated (UTC)
solid-connection-web Error Error Mar 2, 2026 2:07pm
1 Skipped Deployment
Project Deployment Actions Updated (UTC)
solid-connect-web-admin Skipped Skipped Mar 2, 2026 2:07pm

@vercel vercel bot temporarily deployed to Preview – solid-connect-web-admin March 2, 2026 14:04 Inactive
@github-actions github-actions bot added the web label Mar 2, 2026
@coderabbitai
Copy link

coderabbitai bot commented Mar 2, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 422b45c and ed07657.

📒 Files selected for processing (1)
  • apps/web/src/app/university/[homeUniversity]/[id]/page.tsx

Walkthrough

대학교 상세 페이지 컴포넌트에서 간단한 레이아웃 조정이 이루어졌습니다.

  1. 컨테이너의 패딩 제거
    • UniversityDetail을 감싸는 컨테이너의 클래스명 변경
    • "w-full px-5"에서 "w-full"로 수정
    • 수평 패딩(px-5)이 제거되어 더 넓은 화면 활용

이 변경사항은 순수하게 UI/레이아웃 영역에만 영향을 미치며, 데이터 처리 로직이나 네비게이션 동작에는 아무런 영향이 없습니다.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 분

Suggested reviewers

  • wibaek
  • enunsnv
  • khwww
🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed PR 제목이 변경 사항을 명확하게 요약하고 있습니다. 대학 상세 정보 페이지에서 좌우 패딩을 제거했다는 핵심 변화를 정확히 설명합니다.
Description check ✅ Passed PR 설명이 기본적인 구조를 따르고 있지만, 저장소의 템플릿에서 요구하는 '관련 이슈' 섹션이 명확하게 포함되어 있지 않습니다.
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 (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/university-detail-remove-px5

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.

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.

1 participant