[Feat] #110 - navigation 로고를 통합 텍스트 SVG로 교체#110
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (2)
📒 Files selected for processing (1)
📝 WalkthroughWalkthroughNavigation component header logo is updated from separate Ajou icon and text span to a single combined SVG image, with container dimensions adjusted accordingly. No API or behavior changes. ChangesNavigation Logo Update
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~5 minutes Possibly related PRs
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches📝 Generate docstrings
🧪 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 |
🔎 What is this PR?
aim-ajou-logo_text.svg)로 교체합니다. closes [Feat] navigation 로고 교체 #109📝 Changes
src/shared/ui/navigation/navigation.tsx의<img src>를/assets/ajou-logo.svg→/assets/aim-ajou-logo_text.svg로 교체<span>제거h-8 w-8→h-12 w-12로 확대, 불필요한gap-[6px]제거public/assets/aim-ajou-logo_text.svg,public/assets/aim-ajou-logo.svg자산 추가📸 Screenshots (선택)
📚 Background / Context (선택)
✔ Checklist
pnpm build)pnpm lint)🙏 Request
h-12 w-12)나 컨테이너 패딩 조정이 필요하면 코멘트 남겨주세요.Summary by CodeRabbit