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

[Map] 메인 페이지 지도 구현 #107

Merged
merged 27 commits into from
Oct 23, 2023
Merged

[Map] 메인 페이지 지도 구현 #107

merged 27 commits into from
Oct 23, 2023

Conversation

kimeodml
Copy link
Contributor

@kimeodml kimeodml commented Oct 9, 2023

[#102] request

  • 마커 클러스터링 구현 및 모바일 디자인 수정했습니다.
  • 몇가지 기능들을 추가 및 수정했습니다.
    • 로그인 시 사이드바에 프로필 이미지 표시
    • 마커에 표시되는 이미지를 상점 이미지로 변경
    • 검색바 클릭시 검색 페이지로 이동
    • 사이드바 펼쳐진 상태를 기본값으로 변경

Please check if the PR fulfills these requirements

  • It's submitted to develop branch, not the main branch
  • The commit message follows our guidelines
  • There are no warning message when you run yarn lint
  • Docs updated for breaking changes

Screenshot

@kimeodml kimeodml removed the request for review from 20HyeonsuLee October 9, 2023 13:38
@kimeodml kimeodml self-assigned this Oct 9, 2023
@kimeodml kimeodml added enhancement New feature or request UI UI 구현 labels Oct 9, 2023
Copy link
Contributor

@chaeseungyun chaeseungyun left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!
제 환경에서 에러가 하나 있는데 확인 부탁드리겠습니다

src/components/common/SideNavigation/index.tsx Outdated Show resolved Hide resolved
Copy link
Contributor

@hyejun0228 hyejun0228 left a comment

Choose a reason for hiding this comment

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

수고하셨습니다! 궁금한점 리뷰 달아놨습니다.

  • 모바일 버전에서 오른쪽 밑에 있는 네이버 마크가 figma에는 없어서 수정해야할 것 같습니다.

@kimeodml
Copy link
Contributor Author

수고하셨습니다! 궁금한점 리뷰 달아놨습니다.

  • 모바일 버전에서 오른쪽 밑에 있는 네이버 마크가 figma에는 없어서 수정해야할 것 같습니다.

로고 제거 하는 옵션이 있었는데 현재는 그 기능을 빼 버렸더라고요. 그래서 네이버 마크는 유지해야합니다!

@kimeodml kimeodml merged commit 4065df0 into develop Oct 23, 2023
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request UI UI 구현
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants