-
Notifications
You must be signed in to change notification settings - Fork 0
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
[feat] [#37] Map 화면 클러스터링 구현 및 Map Marker, Booth HorizontalPager 연동 #66
Conversation
기본 Material3 색상에서 앱에서 사용하는 색상으로 변경
stroke 추가
…ure/map-screen-ui
…ifest/unifest-android into feature/map-screen-ui
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 수고많으셨습니다!! 푸시가 아니라 리뷰로 남겼어야 했는데 다음부턴 유의하겠습니다
@@ -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) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@wjdtkdgns777
이거 제가 양쪽으로 padding 안준 이유가 사진과 같이 화면내에서 가장 오른쪽에 보이는 chip 이 padding에 의해서 짤리는 것 때문이였거든요. 오른쪽으로 밀어버리면 제일 왼쪽 아이템이 패딩에 의해 잘리는 문제가 생겨서. 베스트는 시작 위치는 왼쪽 끝과 조금 떨어져있지만 남지만 오른쪽으로 슬라이드하면 아이템이 짤리지 않고 왼쪽 끝에 도달할 수 있는 그런 형태입니다.
패딩에 round coner 를 적용해서 좀더 자연스럽게 짤리긴했지만, 근본적으로는 design의 문제라고 생각합니다. 이럴거면 TopAppBar에 round corner 가 있으면 안되거든요
기타)
#55 해결
#57 해결
TODO)