Skip to content

Conversation

@grapefruit13
Copy link
Contributor

유형

  • 기능 구현
  • UI 구현
  • 리팩토링
  • 버그 해결
  • 문서 업데이트
  • 기타( )

작업 내용

  • BetaBadge 구현
  • NavbarSection에서 NavMenuItem 분리
  • color에 transition 3000ms ease in and out 효과 적용

설명 (선택)

  • 컬러에 transition 효과는 기존에 적용되어있지 않아서 이번에 한 번에 적용했는데 혹시 보시고 제외할 부분이 있거나 제외하는게 낫겠다 싶으면 편하게 말씀 주세요~~ 저는 전에 다 기본으로 넣었을 때 예뻤어서 한 번 추가해봤어요

스크린샷

inactive

image

active

image

리뷰 요구사항

  • nav 관련 컴포넌트가 common 하위에 폴더로 안 묶여있는 상태인데, 따로 nav 폴더 만들어서 묶는건 어떤가여 지금 common 아래 컴포넌트가 너무 많아서 조금 산만한 것 같아요
  • Image 태그에서 사용하는 절대경로들 지금까지는 문자열로 그냥 넣었는데, 객체화하면 나중에 이미지 변경할 때 편할 것 같아서 한 번 해봤어요. 괜찮으시면 앞으로 개발하면서 계속 한 객체에 제가 적은 패턴처럼 추가해나갔으면 좋겠는데, 어떻게 생각하시나여?
export const IMAGE_PATH = {
  beta: {
    active: {
      src: '/imgs/beta-active.png',
      alt: 'active beta badge',
    },
    inactive: {
      src: '/imgs/beta-inactive.png',
      alt: 'inactive beta badge',
    },
    WIDTH: 48,
    HEIGHT: 24,
  },
};

@grapefruit13 grapefruit13 self-assigned this Jan 26, 2025
@grapefruit13 grapefruit13 requested a review from plla2 January 26, 2025 15:18
Copy link
Member

@plla2 plla2 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

그러네요 nav관련 컴포넌트들이 다 흩어져있네여 묶는게 좋아보이긴 합니다!
auth 파트에서도 그렇고 이미지 쓰일일 또 생길거같은데 그부분도 경로 객체로 묶어서 사용하는 방향으로 하면 좋을 것 같습니다 auth쪽 하면서 제가 묶을게여!

@grapefruit13
Copy link
Contributor Author

그러네요 nav관련 컴포넌트들이 다 흩어져있네여 묶는게 좋아보이긴 합니다!

넵 다음 pr부터 공통 컴포넌트 카테고리 해당되는 거 있으면 묶을게요

auth 파트에서도 그렇고 이미지 쓰일일 또 생길거같은데 그부분도 경로 객체로 묶어서 사용하는 방향으로 하면 좋을 것 같습니다 auth쪽 하면서 제가 묶을게여!

네~

@grapefruit13 grapefruit13 merged commit 65b1d4c into dev Jan 28, 2025
1 check passed
@grapefruit13 grapefruit13 deleted the THKV-129 branch January 28, 2025 09:41
plla2 added a commit to plla2/NNplanner-FE that referenced this pull request Jan 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants