Skip to content

[Fix] 지도에 상도시장이 즉시 보이도록 줌 축소#520

Merged
chlwhdtn03 merged 1 commit into
developfrom
fix/festival_map_zoom_level
May 12, 2026
Merged

[Fix] 지도에 상도시장이 즉시 보이도록 줌 축소#520
chlwhdtn03 merged 1 commit into
developfrom
fix/festival_map_zoom_level

Conversation

@chlwhdtn03
Copy link
Copy Markdown
Collaborator

Summary

지도에 상도시장이 즉시 보이도록 DEFAULT_ZOOM를 14.5로 낮췄습니다

Describe your changes

Android Studio 2026 05 12 125105@2x

Issue

  • Resolves #

To reviewers

@chlwhdtn03 chlwhdtn03 requested a review from PeraSite May 12, 2026 03:51
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request updates the default zoom level in MapFragmentView.kt from 17.5 to 14.5 to adjust the initial map view. The reviewer suggests using LatLngBounds and CameraUpdate.fitBounds instead of a hardcoded zoom level to ensure the target area is consistently visible across various device screen sizes and resolutions.

private const val DEFAULT_LATITUDE = 37.49517278813046
private const val DEFAULT_LONGITUDE = 126.95661313346206
private const val DEFAULT_ZOOM = 17.5
private const val DEFAULT_ZOOM = 14.5
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

medium

상도시장이 화면에 즉시 보이도록 줌 레벨을 조정한 의도는 적절합니다. 다만, 고정된 줌 레벨(14.5)을 사용하면 기기의 화면 크기나 해상도에 따라 상도시장 영역이 화면에 다 들어오지 않거나, 반대로 너무 넓은 영역이 보일 수 있는 한계가 있습니다.

사용자 기기의 화면 크기에 관계없이 특정 영역(예: 숭실대학교와 상도시장을 모두 포함하는 영역)을 일관되게 보여주려면, LatLngBounds를 정의하고 CameraUpdate.fitBounds(bounds, padding)를 사용하여 카메라 위치를 설정하는 방식을 고려해 보시기 바랍니다. 이는 다양한 안드로이드 기기 환경에서 더 견고한 UX를 제공할 수 있는 방법입니다.

@chlwhdtn03 chlwhdtn03 merged commit c344622 into develop May 12, 2026
2 checks passed
@chlwhdtn03 chlwhdtn03 deleted the fix/festival_map_zoom_level branch May 12, 2026 03:55
@chlwhdtn03 chlwhdtn03 mentioned this pull request May 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants