바텀 시트 스크롤 영역의 양단에 그라데이션 추가 #149
Conversation
| .width(52.dp) | ||
| .height(28.dp), | ||
| ) | ||
| when { |
There was a problem hiding this comment.
분기가 하나 뿐인데 when 문을 쓴 이유를 모르겠어 if 문으로 하면 되지 않을까?
개인적으로 when 문은 분기가 3개 이상일때 쓰는게 맞다고 생각함
| onClick = { viewModel.openDeleteCellDialog(flag = !uiState.isDeleteCellDialogOpened) }, | ||
| ) | ||
| Spacer(modifier = Modifier.width(9.dp)) | ||
| when { |
There was a problem hiding this comment.
분기가 하나 뿐인데 when 문을 쓴 이유를 모르겠어 if 문으로 하면 되지 않을까?
There was a problem hiding this comment.
다른 방식으로 하다가 바꾸는 것을 잊었어 굿
| }, | ||
| .align(Alignment.CenterHorizontally) | ||
| .padding(horizontal = 8.dp) | ||
| .imePadding(), |
There was a problem hiding this comment.
이거 imePadding과 관련된 Window Insets API 관련 이슈하나 올려놨는데 확인해주면 땡큐
#148
There was a problem hiding this comment.
해당 이슈대로 초기 설정을 해주지 않으면 발생하는 문제점이 있어?
There was a problem hiding this comment.
https://developer.android.com/jetpack/compose/layouts/insets?hl=ko
문서에 따르면
'앱이 콘텐츠를 그리는 위치를 완전히 제어할 수 있도록 하려면 다음 설정 단계를 따르세요. 이러한 단계가 없으면 앱이 시스템 UI 뒤에 검은색 또는 단색을 그리거나 소프트웨어 키보드와 동기식으로 애니메이션을 적용하지 않을 수 있습니다.'
라고 적혀 있긴함
그런데 그새 초기 설정 코드가 변한거같네.. 문서가 업데이트 된듯 업데이트 된 버전으로 한번 더 확인해볼게
There was a problem hiding this comment.
"수 있습니다"의 경우를 모르겠으니까 모르고 무턱대고 하라는대로 하는 느낌이들어서 그럴바에는 안 쓰고도 한다 이런 기분..
| viewModel.colorChanged( | ||
| mainColor = it.mainColor, | ||
| subColor = it.subColor, | ||
| datePickerScope = rememberCoroutineScope(), |
There was a problem hiding this comment.
해당 파일 맨위에 선언된 scope 가져다 써도 될듯?
| @@ -190,11 +193,12 @@ fun BandalartBottomSheet( | |||
| } | |||
|
|
|||
| val focusManager = LocalFocusManager.current | |||
| val scrollState = rememberScrollState() | |||
There was a problem hiding this comment.
요런 변수들 맨 윗짝에 다른 변수들 선언된 곳에 같이 두는게 좋을듯? focusManager 도 그렇고
|
checkout 해서 확인해봤을 때 정상적으로 그라데이션이 보이는 것을 확인 |
|
|
scrollState, background를 활용해서 적용