[chore/#316] 지도뷰 진입 시 카메라 이동 애니메이션 삭제#323
Merged
SeungWon1125 merged 4 commits intodevelopfrom Nov 6, 2025
Merged
Conversation
1 task
dudwntjs
requested changes
Nov 6, 2025
Comment on lines
+95
to
+103
| func configureCamera(_ mapView: NMFMapView, coordinator: Coordinator) { | ||
| DispatchQueue.main.async { | ||
| let cameraUpdate = NMFCameraUpdate( | ||
| scrollTo: NMGLatLng(lat: latitude, lng: longitude), | ||
| zoomTo: zoomLevel | ||
| ) | ||
| cameraUpdate.animation = .fly | ||
|
|
||
| if coordinator.currentLatitude == 0.0 && coordinator.currentLongitude == 0.0 { | ||
| cameraUpdate.animation = .none |
Contributor
There was a problem hiding this comment.
Coordinator를 인자로 받는 형태로 변경한 건 좋아여
근디 (0.0, 0.0)으로 초기 여부를 판단하는 것 같은데, didInitialMove Bool 플래그를 쓰면 안되는건가요??
둘이 모가 다른거조
Collaborator
Author
There was a problem hiding this comment.
방금 고쳤는데, CourseDetailMapView는 데이터 배열이 비어있을 때 guard문 처리를 해서 중복 호출을 방지했더라고요 (과거의 제가) 그래서 플래그 형식으로 구현이 가능했던 것이고, PlaceDetailMapView에서는 그런 처리를 하지 않았었기 때문에(과거의 제가) 중복으로 불려서 플래그가 안 먹혔네요
결론
PlaceDetailMapView에 UpdateUIView()가 불릴 때 처음 (0.0, 0.0)으로 초기화 될 때 UI가 업데이트되는 것을 guard로 막고,
이후 (0.0, 0.0) -> 장소 좌표로 이동할 때는 플래그값으로 애니메이션을 .none으로 (CourseDetailMapView방식과 동일) 하도록 수정했어요~!
dudwntjs
approved these changes
Nov 6, 2025
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.

📄 작업 내용
(원래는 저 멀리서부터 날아왔음)
💻 주요 코드 설명
장소 상세 & 코스 상세
첫 애니메이션을 끄는 방식으로 구현했습니다.
첫 애니메이션이 (초기값) -> (특정 코스들의 중심 좌표)로 이동이기 때문에 해당 애니메이션을 끄는 방식으로 수정했어요
했던 고민
장소 상세나, 코스 상세를 fetch하기 전에 지도뷰가 그려지고, fetch된 데이터에서 좌표값을 기반으로 지도에 카메라나 마커를 표시하기 때문에
초기값 -> 우리가 보고싶은 좌표로 이동이 생기게 됩니다.
(원래의 문제는 기본값으로
.fly애니메이션을 적용해서 이 초기값에서 이동하는 과정에도 애니메이션이 생겼던 것입니닷)처음에는 fetch될 때까지 지도를 그리지 말고 데이터를 받음과 동시에 지도를 초기화 할까 고민해 봤지만,
데이터가 느리게 받아와지는 경우 지도가 아에 보이지 않는 상황이 발생해 이 부분에서 오히려 UX를 해친다 생각하여
지도는 보이지만, 데이터가 받아와졌을 때 애니메이션 없이 빠르게 카메라를 이동시키게 했어요
🔗 연결된 이슈