[Campus][Refactor] 메인 화면 식단을 Compose로 리팩토링 #660
Conversation
| diningComposeView.apply { | ||
| setContent { | ||
| KoinTheme { | ||
| val diningData by viewModel.diningData.collectAsState() |
There was a problem hiding this comment.
collectAsStateWithLifecycle을 사용하지 않은 이유가 있나여 ?
|
|
||
| composeOptions { | ||
| kotlinCompilerExtensionVersion = "1.5.10" | ||
| } | ||
|
|
||
| buildFeatures { | ||
| compose = true | ||
| } |
There was a problem hiding this comment.
build-logic에 compose plugin이 정의되어 있는 거로 아는데, 활용되어도 좋을 거 같아여
There was a problem hiding this comment.
library가 아니라 application 모듈용 플러그인이라, 사용이 불가한 것 같습니다!
There was a problem hiding this comment.
아하 나중에 라이브러리용으로도 추가되면 좋겠네여~
| EventLogger.logClickEvent( | ||
| EventAction.CAMPUS, | ||
| AnalyticsConstant.Label.MAIN_MENU_MOVEDETAILVIEW, | ||
| "${ | ||
| context.getString( | ||
| if (type == DiningType.NextBreakfast) R.string.dining_tomorrow else R.string.dining_today | ||
| ) | ||
| } ${context.getString(R.string.navigation_item_dining)}" | ||
| ) |
There was a problem hiding this comment.
이 위치가 맞는 것 같아서 여기에 두었습니다! 혹시 다른 위치가 적합할까요?
There was a problem hiding this comment.
어떤 로깅인지 정확힌 모르겠지만... 리컴포지션마다 실행될 수 있을 것 같아서요!!
LaunchedEffect 같은 곳으로 옮기는 게 어떨까요?
| Text( | ||
| modifier = Modifier.padding(start = 20.dp), | ||
| text = "${ | ||
| context.getString( |
There was a problem hiding this comment.
stringResource를 사용하시지 않은 이유가 있나요 ?
There was a problem hiding this comment.
이건... 밤 새다가 정신줄을 놓았네요
수정하겠습니다!
| .clickable { | ||
| Intent(context, DiningActivity::class.java).let { | ||
| context.startActivity(it) | ||
| } | ||
| }, |
There was a problem hiding this comment.
Card 자체에 있는 onClick 매개변수를 써도 좋을듯해용
| fun DiningContent( | ||
| diningData: Dining, | ||
| selectedType: DiningType, | ||
| modifier: Modifier = Modifier |
close #659
개요
작업 내용