Skip to content

Conversation

@grapefruit13
Copy link
Contributor

@grapefruit13 grapefruit13 commented Jan 12, 2025

유형

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

작업 내용

  • Navbar, NavItem 리디자인 적용
  • 아이콘 추가

설명 (선택)

  • NavItem hover는 피그마에 디자인이 아직 없어서 제외하고 구현했습니다. (후에 hover 디자인 추가되면 수정할게요!)

스크린샷

image
  • 흰글씨 hover
  • 초록글씨 active

리뷰 요구사항

  • typo 컨플릭트 방지를 위해서 이 Pr 먼저 머지한 후에 작업시작하시면 좋을 것 같습니다~
    • hover 스타일 적용을 위해 customTypo에 따로 클래스네임 주었습니다

@grapefruit13 grapefruit13 self-assigned this Jan 12, 2025
@grapefruit13 grapefruit13 marked this pull request as ready for review January 12, 2025 13:16
@grapefruit13 grapefruit13 requested a review from plla2 January 12, 2025 13:17
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.

제가 customTypography 네이밍에 대한 메세지를 너무 늦게봐서.. 제가 오늘 작업할 때 네이밍이랑 사용된 곳 한번에 수정해둘까요? 아니면 각자 선언한 부분 각자 작업 하는게 편하실까요?

width={ICON_SIZE}
height={ICON_SIZE}
/>
<span>{name}</span>
Copy link
Member

Choose a reason for hiding this comment

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

해당 Typography도 hover 디자인 추가되면 추가하실 예정이신가요??

Copy link
Contributor Author

Choose a reason for hiding this comment

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

엇 typo 넣는걸로 바꿀게여 놓쳤네요

Copy link
Contributor Author

Choose a reason for hiding this comment

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

제가 customTypography 네이밍에 대한 메세지를 너무 늦게봐서.. 제가 오늘 작업할 때 네이밍이랑 사용된 곳 한번에 수정해둘까요? 아니면 각자 선언한 부분 각자 작업 하는게 편하실까요?

한 번에 수정해주시면 좋을 것 같습니다! 아직까지는 custom typo 쓴 곳이 많지 않아서여

true: 'bg-green-100 text-green-800 ',
false: 'text-gray-600',
true: 'bg-grey-900 text-green-500',
false: 'text-gray-500',
Copy link
Member

Choose a reason for hiding this comment

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

grey대신 gray로 하신 이유가 있으신가여!?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

엇 오타네요 수정할게여

@grapefruit13 grapefruit13 merged commit 5760cfc into dev Jan 13, 2025
@grapefruit13 grapefruit13 deleted the THKV-118 branch January 13, 2025 02:28
plla2 added a commit to plla2/NNplanner-FE that referenced this pull request Jan 13, 2025
Feat[#THKV-118] : Navbar 리디자인
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