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

[fix] [#157] [#158] [#159] [#160] minor bugs fix #161

Merged
merged 6 commits into from
May 8, 2024
Merged

Conversation

wjdtkdgns777
Copy link
Collaborator

  1. 157번 이슈는 패딩을 추가하면서 발생한 문제로, 흰 사각형이 항상 보여 어색하던 이슈였습니다. 패딩을 지우고 대신 카메라 위치를 조정해 수정해주었습니다
  2. 158번 이슈는 인트로 화면에서 처음으로 진입했을시에는 selectedRegion이 값이 지정되어있지 않아 ""로 지정되어져 아무 축제값도 보여주지못하는 이슈였습니다. UiState에서 기본값으로 전체를 설정해줘 해결했습니다. (다른 지역검색시 selectedRegion이 새로 지정되므로 문제가 발생하지않음)
  3. 159번 이슈는 맵 탑앱바에서의 부스 SearchTextField에서 clearText가 작동하지 않았는데, it.copy(festivalSearchText = TextFieldValue()) 로 작성되어있던 이슈였습니다. 수정했습니다
  4. 160번 이슈는 양옆에 horizontal 패딩을 4dp씩 주어 해결했습니다. 이전 화면인 부스 디테일화면의 앱바에서의 뒤로가기 위치가 같았다면 다른식으로 해결했을텐데 그렇진 않아서 쉽게 해결해뒀습니다. 추가로 부스 상세 위치를 보여주는 지도의 카메라 줌을 수정해주었습니다.
    스크린샷 2024-05-08 212314
    스크린샷 2024-05-08 212835

@wjdtkdgns777 wjdtkdgns777 added the bug Something isn't working label May 8, 2024
@wjdtkdgns777 wjdtkdgns777 self-assigned this May 8, 2024
@wjdtkdgns777 wjdtkdgns777 changed the title [fix] [#157] [#158] [#159] minor bug fix [fix] [#157] [#158] [#159] [#160] minor bugs fix May 8, 2024
@@ -68,7 +68,7 @@ fun BoothLocationScreen(
) {
Box(modifier = Modifier.fillMaxSize()) {
val cameraPositionState = rememberCameraPositionState {
position = CameraPosition(LatLng(uiState.boothDetailInfo.latitude.toDouble(), uiState.boothDetailInfo.longitude.toDouble()), 14.0)
position = CameraPosition(LatLng(uiState.boothDetailInfo.latitude.toDouble(), uiState.boothDetailInfo.longitude.toDouble()), 14.8)
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

요부분 수정했다 원래 로직대로 현재 보려는 마커를 중심으로 하도록 카메라 위치를 조정하게 다시 돌렸는데,
figma를 보니 건국대학교를 중심으로 두는게 맞는거같은데 어떻게 생각하시나요?

Copy link
Collaborator

Choose a reason for hiding this comment

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

네 건대 중심으로 하는게 좋을거같네여

Copy link
Collaborator

@easyhooon easyhooon left a comment

Choose a reason for hiding this comment

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

좋네요! 수고하셨습니다

@wjdtkdgns777 wjdtkdgns777 merged commit 85152eb into develop May 8, 2024
1 check passed
@wjdtkdgns777 wjdtkdgns777 deleted the fix/bug-fix branch May 8, 2024 13:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
2 participants