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

하단 내비게이션 바를 개발합니다. #21

Open
wants to merge 10 commits into
base: develop
Choose a base branch
from

Conversation

jaychang99
Copy link
Member

작성자: @jaychang99

Related to

체크 리스트

  • 적절한 제목으로 수정했나요?
  • 상단에 이슈 번호를 기입했나요?
  • Target Branch를 올바르게 설정했나요?
  • Label을 알맞게 설정했나요?

작업 내역

  • 하단 내비게이션 바를 개발합니다.

image

image

각각 지정된 링크로 이동되며, 메뉴 항목은 상수로 별도 관리됩니다.

비고

@jaychang99 jaychang99 added the feature 신규 컴포넌트/기능 개발 (제일 일반적) label Aug 27, 2023
@jaychang99 jaychang99 self-assigned this Aug 27, 2023
Copy link
Member

@kanghoon-dev kanghoon-dev left a comment

Choose a reason for hiding this comment

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

그리 크지 않은 PR이라 작은 질문입니다!

@jaychang99
Copy link
Member Author

@sera2002 @hoon5083 자체적으로 발견한 이슈 공유드립니다.

현재 의 로직은 url 을 직접비교 하고 있고 startsWith 를 쓴다거나 하고 있지 않습니다.

따라서 예시로 '마이페이지' 같은 경우 /mypage (마이페이지 메인) 에서만 활성화되고, 이외 내 정보 수정이라든가 탈퇴 페이지 등 /mypage/edit, /mypage/delete 등에서는 활성화되지 않습니다. 이 로직을 조금 추가하여야 할 것 같습니다.

@jaychang99
Copy link
Member Author

jaychang99 commented Sep 16, 2023

@hoon5083 @sera2002
말씀주신 사항들 반영과 함께, 버그픽스 및 상단에 움직이는 slider 바를 추가하였습니다.
확인 부탁드립니다!

navbar.mov

그리고 갤럭시 폴드 접혔을 때 등에 대해 저 좁은 viewport width 디바이스에 대해 Navbar 만이라도 대응하기 위해 theme.size.fold 라는 값을 추가했으며, 이 값의 break 기준은 가로 350px 입니다. (참고: 아이폰 12 Pro: 390px)

Copy link
Collaborator

@sera2222 sera2222 left a comment

Choose a reason for hiding this comment

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

수고하셨습니다!

Copy link
Collaborator

Choose a reason for hiding this comment

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

홈에서 맛집이나 단체 등을 눌러서 상세 페이지로 이동하게 되면 Navbar 슬라이더는 어디에 위치하게 되는 건가요?

Copy link
Member Author

Choose a reason for hiding this comment

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

@sera2002 오 ... 예리하시네요 ㅋㅋㅋㅋㅋㅋ 제가 feat/temp-mixture-pages 에서 다 합쳐보고나니까 이 에러가 보이더군요... 외부에서 router 변경 때에도 slider 위치가 바뀔 수 있도록 보완하겠습니다! 감사합니다!

Copy link
Member Author

Choose a reason for hiding this comment

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

@sera2002 외부 라우터 변경에도 navbar 가 바뀔 수 있게 변경하여 보았습니다. 추가로, /organization 등의 url 로 직접 접근 시 slider 가 홈에 잠시 왔다가 다른 곳으로 가는 현상이 있었는데, 이부분도 해결해 보았습니다!

chages 바로보기: https://github.com/SystemConsultantGroup/foodhub-frontend/pull/21/files/d39a11406958476368a7e25bfa5e28c019345fe2..476e74194edfb0f917a08b5c7ee2c44387810355

@kanghoon-dev
Copy link
Member

@sera2002 @hoon5083 자체적으로 발견한 이슈 공유드립니다.

현재 의 로직은 url 을 직접비교 하고 있고 startsWith 를 쓴다거나 하고 있지 않습니다.

따라서 예시로 '마이페이지' 같은 경우 /mypage (마이페이지 메인) 에서만 활성화되고, 이외 내 정보 수정이라든가 탈퇴 페이지 등 /mypage/edit, /mypage/delete 등에서는 활성화되지 않습니다. 이 로직을 조금 추가하여야 할 것 같습니다.

@jaychang99 혹시 이부분 useRouter에서 사용할만한 다른 요소가 있나요? router 객체에는 계층별로 구분하여 들어있진 않는거 같긴하네요.... 그냥 "/"기준으로 split하거나 startWith사용해도 괜찮을 것 같긴 합니다.

Copy link
Member

@kanghoon-dev kanghoon-dev left a comment

Choose a reason for hiding this comment

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

수고하셨습니다! route 관련문제 수정하는대로 바로 머지하셔도 될 듯 합니다

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.

3 participants