[Main] 메인 화면 BottomBar 구조 추가 #26
Conversation
feature/main/src/main/java/com/droidknights/app2023/feature/main/MainScreen.kt
Outdated
Show resolved
Hide resolved
feature/main/src/main/java/com/droidknights/app2023/feature/main/MainScreen.kt
Outdated
Show resolved
Hide resolved
| import androidx.compose.ui.unit.dp | ||
|
|
||
| @Composable | ||
| fun MainScreen() { |
feature/main/src/main/java/com/droidknights/app2023/feature/main/MainScreen.kt
Outdated
Show resolved
Hide resolved
| painter = painterResource(id = resId), | ||
| contentDescription = null, | ||
| tint = if (selected) Color(0xFF49F300) else Color(0xFFDCDCDC), | ||
| modifier = Modifier.size(34.dp), |
There was a problem hiding this comment.
icon 사이즈를 늘리는것과 box의 사이즈 늘리는게 좀 달라서
기본 Icon은 내부적으로 24.dp를 강제로 셋팅합니다.
There was a problem hiding this comment.
Icon 컴포저블은 painter의 크기를 가져올 수 없는 경우에만 24로 고정합니다.
| color = Color(0xFFDCDCDC), // lightgray | ||
| shape = RoundedCornerShape(size = 28.dp) | ||
| ) | ||
| .background(color = Color(0xFFFFFFFF), shape = RoundedCornerShape(28.dp)) |
There was a problem hiding this comment.
Color 정보 테마 지정하면서 지정하는게 좋겠네요.
There was a problem hiding this comment.
아직 디자인 정의가 안되어있는 상태입니다.
따로 이슈 생성 해 두었습니다
| .navigationBarsPadding() | ||
| .padding(start = 8.dp, end = 8.dp, bottom = 28.dp) | ||
| .fillMaxWidth() | ||
| .height(56.dp) |
There was a problem hiding this comment.
modifier를 외부에서 주입하고있긴한데 내부에서 padding 지정과 외부에서도 padding과 같은 지정을 할 수 있어서
사실 좋은 방법은 아니에요.
그리고 modifier의 순서는 매우 중요하지만 여기서는 fil/height가 먼저와도 상관은 없어요. padding이 어차피 색상 먹기 전에만 지정하면 아웃사이드의 padding이고, 색상 먹고 난 다음엔 inside의 padding이라서
There was a problem hiding this comment.
xml에서 layout_width layout_height을 최상단에 쓰듯이 이 속성들도 가장 앞에 위치하는게 눈에 잘 들어오네요
There was a problem hiding this comment.
현재는 Modifier가 외부에서 주입이 필요하지 않으니 제거해두겠습니다.
098c5cb
Issue
Overview (Required)
Etc
Screenshot