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] [#79] Snackbar 위치 문제 해결 및 Booth 상세 화면 디자인 디테일 보완 #87

Merged
merged 4 commits into from
Apr 19, 2024

Conversation

easyhooon
Copy link
Collaborator

@easyhooon easyhooon commented Apr 19, 2024

BoothDetail 화면 내에서는 MainScreen의 Scaffold 내에 SnackBar 를 사용하는 것이 아닌, BoothDetail 내에 Snackbar 를 사용하는 것으로 문제 해결

  • 해당 화면에서만 특수하게 bottomPadding 을 필요로 하기 때문
  • Scaffold 를 사용하지 않아도 Snackbar 사용 가능합니다
  • 다른 화면들의 경우 MainScreen 의 Scaffold 내에 Snackbar 사용

BoothDetail 화면 내 디자인 디테일 보완

BoothDetail 내에 Snackbar 별도 처리하는 방식으로
@easyhooon easyhooon added bug Something isn't working design tasks releated to design labels Apr 19, 2024
@easyhooon easyhooon requested a review from wjdtkdgns777 April 19, 2024 00:24
@easyhooon easyhooon self-assigned this Apr 19, 2024
@easyhooon easyhooon linked an issue Apr 19, 2024 that may be closed by this pull request
@easyhooon easyhooon changed the title [Fix] [#79] Snackbar 위치 문제 해결 및 Booth 상세 화면 디자인 디테일 보완 [fix] [#79] Snackbar 위치 문제 해결 및 Booth 상세 화면 디자인 디테일 보완 Apr 19, 2024
@wjdtkdgns777
Copy link
Collaborator

오 스낵바 시간을 저렇게 해결하셨군요 굿..

Box(modifier = Modifier.fillMaxSize()) {
BoothDetailContent(
uiState = uiState,
onNavigateToBoothLocation = onNavigateToBoothLocation,
bottomPadding = 116.dp,
bottomPadding = padding.calculateBottomPadding(),
Copy link
Collaborator

Choose a reason for hiding this comment

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

현재 바텀바에 컨텐츠가 가려보이는 문제가있는데 해당부분이 문제인것같습니다.
이부분이 실제 바텀바의 높이와 달리 계산되는지 bottomPadding = 116.dp,으로 작성하면 정상작동합니다

Copy link
Collaborator Author

@easyhooon easyhooon Apr 19, 2024

Choose a reason for hiding this comment

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

현재 바텀바에 컨텐츠가 가려보이는 문제가있는데 해당부분이 문제인것같습니다.
이부분이 실제 바텀바의 높이와 달리 계산되는지 bottomPadding = 116.dp,으로 작성하면 정상작동합니다

스크롤을 지원하는 화면이라 가리는거 자체가 문제가 되진 않는다고 생각했는데 반영하겠습니다. 혹시 어떤식으로 보이나여? 사진 보여주실수있는지

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

아 마지막까지 스크롤 했을 때 문제가 발생하겠군요!

Copy link
Collaborator

Choose a reason for hiding this comment

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

스크린샷 2024-04-19 170935

넵 맞습니다

맨 마지막 아이템이 바텀바에 의해 가려지는 문제 해결
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 merged commit ac8105e into develop Apr 19, 2024
1 check passed
@easyhooon easyhooon deleted the fix/snackbar-location branch April 19, 2024 08:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working design tasks releated to design
Projects
None yet
Development

Successfully merging this pull request may close these issues.

SnackBar의 화면별 위치 설정 추가 필요
2 participants