[ADD/#32] 공고 정렬 바텀시트 구현 #45
Conversation
| import androidx.compose.material3.rememberModalBottomSheetState | ||
| import androidx.compose.runtime.Composable | ||
| import androidx.compose.ui.Modifier | ||
| import com.terning.core.designsystem.theme.White | ||
|
|
||
| @OptIn(ExperimentalMaterial3Api::class) | ||
| @Composable | ||
| fun TerningBasicBottomSheet( | ||
| modifier: Modifier = Modifier, |
There was a problem hiding this comment.
이거 사용 안해서 삭제할게욤!!
| .padding(horizontal = 24.dp), | ||
| verticalArrangement = Arrangement.spacedBy(8.dp) | ||
| ) { | ||
| items(5) { sortIndex -> |
There was a problem hiding this comment.
전 이런 부분에 const val, companion object 등을 사용하여 하드코딩을 피하는 편입니다!!
그러는 편이 코드 보기에 직관적일 것 같아요!!
There was a problem hiding this comment.
오호 그게 훨씬 더 좋아보이네요!! 감사합니다:)
| enum class SortBy(@StringRes val type: Int) { | ||
| EARLIEST(R.string.sort_by_earliest), | ||
| SHORTEST(R.string.sort_by_shortest), | ||
| LONGEST(R.string.sort_by_longest), | ||
| SCRAP(R.string.sort_by_scrap), | ||
| VIEW_COUNT(R.string.sort_by_view_count), | ||
| } |
There was a problem hiding this comment.
파라미터로 들어가는 문자열 리소스 값들이 코드에서 직접적으로 사용되나요??
There was a problem hiding this comment.
넵!! 바텀시트랑 버튼에서 사용합니다!
| package com.terning.core.designsystem.component.bottomsheet | ||
|
|
||
| import androidx.annotation.StringRes | ||
| import com.terning.core.R | ||
|
|
||
| enum class SortBy(@StringRes val type: Int) { |
There was a problem hiding this comment.
홈에서 쓰이는 부분을 core 모듈에서 관리하는 이유가 있나용?
이 부분은 TerningBasicBottomSheet를 재사용하는 부분이 아니라, 홈의 바텀시트에 들어갈 구성이라면 feature 모듈에서 관리해줘도 될 것 같아서요!
There was a problem hiding this comment.
이게 탐색 뷰에서도 동일한 바텀시트가 사용돼서 홈, 탐색 뷰에서 같이 사용하려고 이렇게 구현했는데 저도 이 enum class 위치가 여기가 맞는지 고민이네요...🥹 이건 좀 더 고민해볼게요..!!
|
|
||
| @OptIn(ExperimentalMaterial3Api::class) |
There was a problem hiding this comment.
저번에 얘기해봤을 때 요거 라이브러리 추가로 없앨 수 있지 않았나용,,? 괜찮다면 그걸루 해봐도 될 것 같네영
There was a problem hiding this comment.
근데 그게 추가를 해도 없애면 오류가 생기더라구요🥹 이것도 다른 방법 찾게 되면 한번 해볼게요!!
| modifier: Modifier = Modifier, | ||
| onDismiss: () -> Unit, | ||
| currentSortBy: Int, | ||
| newSortBy: MutableState<Int> = mutableStateOf(currentSortBy), |
There was a problem hiding this comment.
| modifier: Modifier = Modifier, | |
| onDismiss: () -> Unit, | |
| currentSortBy: Int, | |
| newSortBy: MutableState<Int> = mutableStateOf(currentSortBy), | |
| onDismiss: () -> Unit, | |
| currentSortBy: Int, | |
| modifier: Modifier = Modifier, | |
| newSortBy: MutableState<Int> = mutableStateOf(currentSortBy), |
이렇게 순서 맞춰주면 좋을 것 같아요!
| .noRippleClickable { | ||
| newSortBy.value = sortIndex | ||
| scope | ||
| .launch { sheetState.hide() } | ||
| .invokeOnCompletion { | ||
| if (!sheetState.isVisible) { | ||
| onDismiss() | ||
| } |
There was a problem hiding this comment.
바텀시트 내려가는 부분 구현해준 효빈언니 최고!!!!!! 참고하겠습니당
| }, | ||
| sheetState = sheetState, | ||
| modifier = modifier.navigationBarsPadding() | ||
| containerColor = White, |
⛳️ Work Description
📸 Screenshot
Screen_Recording_20240711_032836_Terning-Android.mp4
📢 To Reviewers