[Feat] 바텀네비게이션 세팅#38
Conversation
|
수고하셨습니당!! |
| Column( | ||
| modifier = | ||
| Modifier | ||
| .noRippleClickable(onClick = onClick) |
There was a problem hiding this comment.
이 부분은 디자인 상 확정된건지 궁금합니다!
(모달에 버튼도 리플효과를 제거했다가 다시 적용을 했습니다!)
There was a problem hiding this comment.
괜찮다고 합니당! 그대로 진행할게요
giovannijunseokim
left a comment
giovannijunseokim
left a comment
There was a problem hiding this comment.
궁금한 점들 남겨두었습니다~! 추후 확인 부탁드려요. 😁
There was a problem hiding this comment.
p3: Navigator가 정확히 무엇인지 궁금합니다! 👀
| @Serializable | ||
| data object Explore : MainTabRoute | ||
|
|
||
| fun NavController.navigateToExplore(navOptions: NavOptions? = null) { | ||
| navigate(Explore, navOptions) | ||
| } | ||
|
|
||
| fun NavGraphBuilder.exploreNavGraph(paddingValues: PaddingValues) { | ||
| composable<Explore> { | ||
| ExploreRoute(paddingValues) | ||
| } | ||
| } |
There was a problem hiding this comment.
p2: 이 코드가 여기에 존재한다면, 추후 feature 모듈별로 서로 알아야 하는 상황이 생기지 않을까요?
이 부분을 core 내에 작성하면 추후 feature 별로 모듈이 나눠졌을 때 서로 몰라도 이동이 가능할 것 같아서요. 🤔
| @Composable | ||
| fun ExploreRoute(paddingValues: PaddingValues) { | ||
| ExploreScreen( | ||
| modifier = Modifier.padding(paddingValues) | ||
| ) | ||
| } | ||
|
|
||
| @Composable | ||
| private fun ExploreScreen(modifier: Modifier) { |
There was a problem hiding this comment.
p3: Route와 Screen의 차이가 정확히 무엇인지 잘 이해가 안 되어서요. Route는 공개되고 Screen은 공개되지 않는다는 차이가 있는걸까요?
| @Composable | ||
| fun ExploreRoute(paddingValues: PaddingValues) { | ||
| ExploreScreen( | ||
| modifier = Modifier.padding(paddingValues) | ||
| ) | ||
| } |
There was a problem hiding this comment.
p3: PaddingValues를 넘겨주는 이유가 궁금합니다!
| import kotlinx.collections.immutable.toImmutableList | ||
|
|
||
| @Composable | ||
| internal fun MainBottomBar( |
There was a problem hiding this comment.
p2: 현재 단일 모듈인 것 같은데, internal 키워드가 들어간 이유는 무엇인가요?
| internal fun MainBottomBar( | ||
| visible: Boolean, | ||
| tabs: ImmutableList<MainTab>, | ||
| currentTab: MainTab?, |
There was a problem hiding this comment.
p3: currentTab이 null인 경우는 어떤 때 인가요?
만약 현재 Route가 MainTabRoute가 아닌 경우라면 visible을 currentTab != null로 활용해볼 수도 있을 것 같아서요. 👀
| AnimatedVisibility( | ||
| visible = visible, | ||
| enter = fadeIn() + slideIn { IntOffset(0, it.height) }, | ||
| exit = fadeOut() + slideOut { IntOffset(0, it.height) } | ||
| ) { |
There was a problem hiding this comment.
p3: 이 부분은 디자이너분들과 논의가 된 부분인지 궁금합니다!
| @Composable | ||
| private fun RowScope.MainBottomBarItem( | ||
| tab: MainTab, | ||
| selected: Boolean, | ||
| onClick: () -> Unit | ||
| ) { |
There was a problem hiding this comment.
p3: 저한테는 이렇게 함수 분리를 하면 코드를 이동하면서 확인해야 해서 가독성이 떨어진다고 느껴졌는데요. 🤔
재사용되지 않는 코드의 경우 어떤 기준으로 함수 분리를 하시는지 궁금해요. 👀
| .map { it.destination } | ||
| .stateIn( | ||
| scope = coroutineScope, | ||
| started = SharingStarted.WhileSubscribed(5_000), |
There was a problem hiding this comment.
p3: 해당 속성을 사용해주신 이유가 궁금합니다!
… into feat/#83-profile-keyword-graph-component
… into feat/#83-profile-keyword-graph-component
[Feat/#38] 프로필 취향 키워드 칩 컴포넌트 구현
5 participants
📮 관련 이슈
📌 작업 내용
📸 스크린샷
Screen_recording_20260112_193721.mp4
😅 미구현
🫛 To. 리뷰어