-
Notifications
You must be signed in to change notification settings - Fork 0
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
base: develop
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
그리 크지 않은 PR이라 작은 질문입니다!
@sera2002 @hoon5083 자체적으로 발견한 이슈 공유드립니다. 현재 의 로직은 url 을 직접비교 하고 있고 따라서 예시로 '마이페이지' 같은 경우 |
@hoon5083 @sera2002 navbar.mov그리고 갤럭시 폴드 접혔을 때 등에 대해 저 좁은 viewport width 디바이스에 대해 Navbar 만이라도 대응하기 위해 theme.size.fold 라는 값을 추가했으며, 이 값의 break 기준은 가로 350px 입니다. (참고: 아이폰 12 Pro: 390px) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
수고하셨습니다!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
홈에서 맛집이나 단체 등을 눌러서 상세 페이지로 이동하게 되면 Navbar 슬라이더는 어디에 위치하게 되는 건가요?
There was a problem hiding this comment.
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 위치가 바뀔 수 있도록 보완하겠습니다! 감사합니다!
There was a problem hiding this comment.
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 가 홈에 잠시 왔다가 다른 곳으로 가는 현상이 있었는데, 이부분도 해결해 보았습니다!
@jaychang99 혹시 이부분 useRouter에서 사용할만한 다른 요소가 있나요? router 객체에는 계층별로 구분하여 들어있진 않는거 같긴하네요.... 그냥 "/"기준으로 split하거나 startWith사용해도 괜찮을 것 같긴 합니다. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
수고하셨습니다! route 관련문제 수정하는대로 바로 머지하셔도 될 듯 합니다
작성자: @jaychang99
Related to
체크 리스트
작업 내역
각각 지정된 링크로 이동되며, 메뉴 항목은 상수로 별도 관리됩니다.
비고