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.

Sign up for GitHub

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

Jump to bottom

[feat] [#37] Map 화면 클러스터링 구현 및 Map Marker, Booth HorizontalPager 연동 #66

Merged
merged 18 commits into from
Apr 12, 2024
Merged

[feat] [#37] Map 화면 클러스터링 구현 및 Map Marker, Booth HorizontalPager 연동 #66

merged 18 commits into from
Apr 12, 2024

Conversation

easyhooon
Copy link
Collaborator

@easyhooon easyhooon commented Apr 11, 2024 •
edited
Loading

Map화면 기본 Map화면 인기 부스 Map 화면 클러스터링 부스
  • Map 내의 마커를 누를면, 하단의 HorizontalPager 에 해당 아이템의 정보가 띄워지도록 클릭 이벤트를 수정했습니다.
    • 클러스터링된 마커일 경우, 복수개의 아이템, 단일 마커일 경우 단일 아이템
  • HorizontalPager 내에 각각의 부스의 대한 아이템을 누르면 부스 상세화면으로 이동하도록 수정했습니다.
  • 인기 부스 아이콘 버튼을 누르면 인기 부스(Top 5)가 보여지도록 구현했습니다.
  • 다시 인기 부스 아이콘을 누르면 부스 아이템들이 사라지고, 마커를 누를 경우 아이템이 교체됩니다.

기타)

#55 해결
#57 해결

TODO)

  • 같은 속성의 Marker 들만 클러스터링 되도록 -> 클러스터링 알고리즘 자체를 수정해줘야함. 큰 공수 예상
  • 클러스터링 마커 커스텀 -> 위 작업과 연관, 마찬가지로 큰 공수 예상
  • 지도 중앙 위치 조정 -> 대학교가 좀 더 화면의 중심에 위치하도록, 현재는 약간 위로 치우쳐서 있는 것으로 보임. TopAppBar의 CornerRadius 를 반영하기 위해, Map이 TopAppBar의 영역을 완전히 포함하고 있는데, TopAppBar가 뚱뚱해서 상대적으로 지도의 윗부분을 많이 가리기 때문.
  • 인기부스 아이템들이 보여질때, Map 내에 마커를 누를 경우, 자연스럽게 아이템이 변경되는 애니메이션을 넣어주면, 아이템이 교체되었다는 것을 더 직관적으로 알 수 있을 것 같다는 생각이 듬

@easyhooon
[fix] [#55] TextField 키보드가 한번 내려갔을 때, 다시 키보드가 올라오지 않는 문제 해결
5caa3ac
@easyhooon
[style] TextFieldSelectionColor 변경
3d985e2 
기본 Material3 색상에서 앱에서 사용하는 색상으로 변경
@easyhooon
[fix] 부스 상세 화면에 진입 후 모든 화면에 status bar 영역의 icon 이 보이지 않는 이슈 해결
9554eb0
@easyhooon
[fix] [#57] 부스 상세 화면 system bar 영역에 padding 이 생기는 이슈 해결
a22bc99
@easyhooon
[fix] 부스 상세 화면 TopAppBar 상단에 고정 및 padding 값 적용
ee5466b
@easyhooon
[fix] [#57] 스낵바 위치 조정
d60a1a9
@easyhooon
[style] BoothDetail 화면 뒤로가기 아이콘 교체
2d4e4a0 
stroke 추가
@easyhooon
[feat] BoothItem 을 눌러야 Booth 로 이동 하도록 변경
ac9ddb9
@easyhooon
[feat] 랭킹 순위 뱃지 구현
36150df
@easyhooon
[feat] 인기 부스 확장 축소 아이콘 버튼 및 애니메이션 추가
e5f395e
@easyhooon
[feat] 지도 클러스터링 구현 및 Marker 와 Booth HorizontalPager 연동
0e32848
@easyhooon
[chore] code style check success
a8f7b41
@easyhooon easyhooon added enhancement New feature or request design tasks releated to design labels Apr 11, 2024
@easyhooon easyhooon self-assigned this Apr 11, 2024
This was linked to issues Apr 11, 2024
Map 화면 구현 #37
Closed
TextField 키보드가 한번 내려갔을 때, 다시 키보드가 올라오지 않는 문제 #55
Closed
Booth화면 스낵바가 노출될 때, 하단 버튼들을 가리는 문제 #57
Closed
@easyhooon easyhooon changed the title Map 화면 클러스터링 구현 및 Map Marker, Booth HorizontalPager 연동 [feat] [#37] Map 화면 클러스터링 구현 및 Map Marker, Booth HorizontalPager 연동 Apr 11, 2024
@wjdtkdgns777
Copy link
Collaborator

wjdtkdgns777 commented Apr 12, 2024 •
edited by easyhooon
Loading

부스 위치 화면에 진입 후에도 status bar 영역의 icon 이 보이지 않는 이슈가 있어 고쳐서 푸시해두겠습니다

wjdtkdgns777
wjdtkdgns777 approved these changes Apr 12, 2024
View reviewed changes
Copy link
Collaborator

@wjdtkdgns777 wjdtkdgns777 left a comment

Choose a reason for hiding this comment

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

LGTM 수고많으셨습니다!! 푸시가 아니라 리뷰로 남겼어야 했는데 다음부턴 유의하겠습니다

easyhooon
easyhooon commented Apr 12, 2024
View reviewed changes
feature/map/src/main/kotlin/com/unifest/android/feature/map/MapScreen.kt
@@ -309,7 +309,11 @@ fun MapTopAppBar(
Spacer(modifier = Modifier.height(10.dp))
BoothFilterChips(
onChipClick = {},
modifier = Modifier.padding(start = 22.dp),
modifier = Modifier
.padding(horizontal = 10.dp)
Copy link
Collaborator Author

@easyhooon easyhooon Apr 12, 2024 •
edited
Loading

Choose a reason for hiding this comment

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

image

@wjdtkdgns777
이거 제가 양쪽으로 padding 안준 이유가 사진과 같이 화면내에서 가장 오른쪽에 보이는 chip 이 padding에 의해서 짤리는 것 때문이였거든요. 오른쪽으로 밀어버리면 제일 왼쪽 아이템이 패딩에 의해 잘리는 문제가 생겨서. 베스트는 시작 위치는 왼쪽 끝과 조금 떨어져있지만 남지만 오른쪽으로 슬라이드하면 아이템이 짤리지 않고 왼쪽 끝에 도달할 수 있는 그런 형태입니다.

패딩에 round coner 를 적용해서 좀더 자연스럽게 짤리긴했지만, 근본적으로는 design의 문제라고 생각합니다. 이럴거면 TopAppBar에 round corner 가 있으면 안되거든요

@easyhooon easyhooon merged commit e196b0d into develop Apr 12, 2024
1 check passed
@easyhooon easyhooon deleted the feature/map-screen-ui branch April 12, 2024 07:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@wjdtkdgns777 wjdtkdgns777 wjdtkdgns777 approved these changes

Assignees

@easyhooon easyhooon

Labels
design tasks releated to design enhancement New feature or request
Projects
None yet
Milestone
No milestone
2 participants
@easyhooon @wjdtkdgns777