Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[FEAT] Top-Nav 구현 #8

Merged
merged 10 commits into from
Apr 29, 2024
Merged

[FEAT] Top-Nav 구현 #8

merged 10 commits into from
Apr 29, 2024

Conversation

AAminha
Copy link
Member

@AAminha AAminha commented Apr 29, 2024

1️⃣ 요약

3️⃣ 참고사항

  • 로고를 클릭하면 메인페이지로 이동하도록 구현했습니다.
  • TopNavigation은 fixed로 구현하여 스크롤할 때도 고정적으로 보이도록 구현했습니다.
  • PlainButton 컴포넌트(피그마에서는 button이라고 되어있음)를 구현했습니다.
    • 모바일 버전에서의 SideNavigation의 '로그인', '마이페이지', '로그아웃' 버튼에 사용되었습니다.

image

  • SmallButton 컴포넌트(피그마에서는 button-s라고 되어있음)를 구현했습니다.
    • TopNavigation의 메뉴 목록과 로그인 버튼에 사용되었습니다.

image

구현 화면

Vite.+.React.+.TS.-.Chrome.2024-04-29.19-28-20.mp4
Vite.+.React.+.TS.-.Chrome.2024-04-29.19-26-31.mp4

4️⃣ 추후 작업

  • 마이페이지, 로그인, 로그아웃 버튼의 기능을 추가해야 합니다.
    • 로그인, 마이페이지 path가 정해지면 navigate(path)로 추가하면 될 것 같습니다.
    • 로그아웃 기능을 추가해야 합니다.
  • 로그인 여부에 따라 버튼이 다르게 보여지도록 구현해야 합니다.
    • 로그인 로직이 완성되면 작업해야 할 것 같습니다.

@AAminha AAminha added the ✨Feature 기능 개발 이슈 label Apr 29, 2024
@AAminha AAminha self-assigned this Apr 29, 2024
@AAminha AAminha requested a review from mungjin01 April 29, 2024 10:31
Copy link
Collaborator

@mungjin01 mungjin01 left a comment

Choose a reason for hiding this comment

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

LGTM 수고하셨습니다!!

@AAminha AAminha merged commit a34d8d7 into develop Apr 29, 2024
@AAminha AAminha deleted the feat/#3/top-nav branch April 29, 2024 16:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨Feature 기능 개발 이슈
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FEAT] Top-Nav 구현
2 participants