Skip to content

Conversation

@gonn-i
Copy link
Member

@gonn-i gonn-i commented Nov 28, 2025

✨ 변경 사항

  • 해당 메뉴의 텍스트 버튼 호버시에만 텍스트 버튼 색상 primary/60으로 활성화
  • 기본 상태: 텍스트 버튼은 neutral/20
  • 기존의 active 상태 삭제

✅ 체크리스트

  • 기능 동작 확인
  • 코드 리뷰 반영
  • 테스트 통과
  • UI/UX 확인

📸 스크린샷 (선택)

image

📝 기타 참고 사항

Summary by CodeRabbit

릴리스 노트

  • 스타일
    • 헤더 네비게이션의 활성 상태 강조 표시를 제거했습니다.
    • 모든 네비게이션 항목이 이제 통일된 스타일로 표시되며, 호버 효과가 적용됩니다.

✏️ Tip: You can customize this high-level summary in your review settings.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Nov 28, 2025

개요

헤더 네비게이션 컴포넌트의 경로 기반 활성 상태 로직이 제거되었습니다. usePathname 훅, pathname 변수, 조건부 isActive 검사가 모두 삭제되었으며, 네비게이션 항목들은 이제 호버 스타일이 포함된 정적 클래스명으로 렌더링됩니다.

변경 사항

코호트 / 파일 변경 요약
헤더 컴포넌트 단순화
apps/web/src/shared/components/header/index.tsx
usePathname 임포트 제거, pathname 변수 삭제, 경로 기반 활성 상태 조건문 제거. 모든 네비게이션 항목이 일관된 클래스명(호버 스타일 포함)으로 렌더링되도록 통일됨.

코드 리뷰 추정 난이도

🎯 1 (간단함) | ⏱️ ~3분

  • 단일 파일 변경으로 범위가 명확함
  • 기존 기능 제거로 인한 직관적인 리뷰
  • 조건부 로직 제거로 인한 코드 단순화

🐰 경로 추적은 내려놓고, *
항상 반짝이는 호버만 남기네, *
네비게이션이 더 가벼워졌어, *
여정은 간단해지고, *
코드는 자유로워졌네! ✨

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed PR 제목은 GNB 텍스트 버튼 active 인터랙션 삭제라는 주요 변경 사항을 명확하게 설명하고 있으며, 실제 코드 변경(usePathname 제거, active 상태 로직 삭제)과 일치합니다.
Description check ✅ Passed PR 설명이 필수 템플릿 구조(변경 사항, 체크리스트, 스크린샷, 참고 사항)를 완벽하게 따르고 있으며, 모든 항목이 완성되고 상세하게 작성되어 있습니다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feature/Y26W2-446-gnb

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 3648427 and bbfa67e.

📒 Files selected for processing (1)
  • apps/web/src/shared/components/header/index.tsx (1 hunks)
🧰 Additional context used
🧬 Code graph analysis (1)
apps/web/src/shared/components/header/index.tsx (1)
packages/ui/src/index.ts (1)
  • cn (120-120)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)
  • GitHub Check: deploy
  • GitHub Check: build
🔇 Additional comments (1)
apps/web/src/shared/components/header/index.tsx (1)

39-49: Remove contradictory accessibility suggestion that conflicts with PR objective.

The code implementation is correct and matches the stated PR objective to remove active interaction styling from the GNB. The component intentionally does not use usePathname or track active state, using only base styling (text-neutral-20) and hover state (hover:text-primary-60).

The original review's suggestion to add aria-current="page" contradicts this design decision—it would require re-introducing usePathname and active state detection that were deliberately removed. Since the PR checklist confirms UI/UX review was completed, the removal of active state styling appears to be an intentional design decision.

No changes needed.


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.

@github-actions
Copy link

github-actions bot commented Nov 28, 2025

🎫 관련 Jira 티켓 🔗: Y26W2-446

📝 Jira 티켓 정보:

  • 제목: [FE] GNB 텍스트 버튼 인터랙션 상이
  • 상태: 검토 중
  • 담당자: 미지정👻

@github-actions
Copy link

This pull request (commit bbfa67e) has been deployed to Vercel ▲ - View GitHub Actions Workflow Logs

Name Link
🌐 Unique https://ssok-lyhvcn71h-sudosubin-team.vercel.app
🔍 Inspect https://vercel.com/sudosubin-team/ssok/GjACFDY9fckAyDqPxZJ9FBQaTmbo

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants