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

[FE] Refactor/#513 지도 생성 및 핀 생성 페이지 지도 위치 조정 #514

Merged
merged 3 commits into from
Sep 27, 2023

Conversation

semnil5202
Copy link
Collaborator

@semnil5202 semnil5202 commented Sep 27, 2023

작업 대상

  • 지도생성 및 핀 생성 페이지에서 지도 상하가 반전되어 나오는 정책을 변경한다. (지도: 상 , 사이드바: 하 로 모든 페이지를 고정한다.)
  • 핀 생성 페이지 접속 시 장소 이름으로 오토 포커스 되는 기능을 제거한다.
  • 지도 단일 조회 이후 지도 생성, 핀생성 페이지 이동 시 기존에 조회한 핀이 잔류하던 문제를 해결한다.

📄 작업 내용

  • 지도생성 및 핀 생성 페이지에서 지도 상하가 반전되어 나오는 정책을 변경한다. (지도: 상 , 사이드바: 하 로 모든 페이지를 고정한다.)
  • 핀 생성 페이지 접속 시 장소 이름으로 오토 포커스 되는 기능을 제거한다.
  • 지도 단일 조회 이후 지도 생성, 핀생성 페이지 이동 시 기존에 조회한 핀이 잔류하던 문제를 해결한다.

🙋🏻 주의 사항

스크린샷

📎 관련 이슈

close #513

레퍼런스

@semnil5202 semnil5202 self-assigned this Sep 27, 2023
@semnil5202 semnil5202 added FE 프론트엔드 관련 이슈 우선순위 : 상 refactor 리팩토링 관련 이슈 labels Sep 27, 2023
Copy link
Collaborator

@jiwonh423 jiwonh423 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

@GC-Park GC-Park left a comment

Choose a reason for hiding this comment

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

먼저 설명해주셔서 코드 이해 아주 수월하게 되었습니다~!

바로 approve 합니다 굿~!

if ($isAddPage) return 'column';
if ($layoutWidth === '372px') return 'column-reverse';
Copy link
Collaborator

Choose a reason for hiding this comment

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

모두 하단으로 배치 시켰군요 굿 !

Comment on lines +232 to +235
if (!topicId && markers && markers.length > 0) {
removeMarkers();
removeInfowindows();
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

이 코드가 방금 말씀하신 핀 생성하기 갔을 때 핀 찍혀있는거 초기화 하는 부분이군요 ! 굿~!

Comment on lines +144 to +147
if (!pulledPinIds && markers && markers.length > 0) {
removeMarkers();
removeInfowindows();
}
Copy link
Collaborator

Choose a reason for hiding this comment

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

same thing~

@semnil5202 semnil5202 merged commit 0c5bf0d into develop-FE-2 Sep 27, 2023
1 check passed
@semnil5202 semnil5202 deleted the refactor/#513 branch September 27, 2023 08:28
@yoondgu yoondgu added this to the 최종 데모데이 milestone Oct 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
FE 프론트엔드 관련 이슈 refactor 리팩토링 관련 이슈 우선순위 : 상
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants