-
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
Feature/#76 메인페이지 마커 디자인변경, 드롭다운메뉴 제작 #80
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.
휴가를 좀 다녀오느라 리뷰가 늦어졌네요 😅 몇가지 수정사항 확인해주세요~~
src/pages/Home/components/Map/components/MobileOptions/MobileOptions.module.scss
Outdated
Show resolved
Hide resolved
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.
고생하셨습니다!
저는 큰 문제를 못찾아서 수정 사항 완료되면 approve 하겠습니다
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.
위 코멘트대로 defaultImg
쪽만 바뀌면 병합해도 될 것 같아요~ 고생하셨어여
[#76] request
1차베포버전 메인페이지 디자인 수정했습니다.
마커 디자인변경하고, 마커필터 드롭다운메뉴로 구성변경했습니다.
마커에 인덱스 번호를 추가하는 과정에서 useEffect와의 충돌로 인덱싱에 문제가 생겨서 마커의 z-index 속성을 이용해서 getZIndex를 사용했습니다.
따로 마커의 인덱스 번호를 방법이 있다면 해당 방법으로 수정하도록 하겠습니다.
Please check if the PR fulfills these requirements
develop
branch, not themain
branchyarn lint
Screenshot