[Feature] 버스 도메인 리뉴얼 #516
[Feature] 버스 도메인 리뉴얼 #516
Conversation
| @@ -4,10 +4,12 @@ | |||
| <application> | |||
| <activity | |||
| android:name="in.koreatech.bus.BusTimetableActivity" | |||
| android:hardwareAccelerated="true" | |||
There was a problem hiding this comment.
Target API 14이상 부터 hardwareAccelerated 값이 default로 true라고 하는데, 여기서 명시적으로 선언해준 이유가 있을까요? 🤔
참고 : https://developer.android.com/topic/performance/hardware-accel
There was a problem hiding this comment.
아뇽 몰랐어요 ㅎㅎ!! 감사해요!!
data/src/main/java/in/koreatech/koin/data/di/datastore/DataStoreModule.kt
Show resolved
Hide resolved
feature/bus/src/main/java/in/koreatech/bus/screen/search/composable/BusSearchView.kt
Show resolved
Hide resolved
feature/bus/src/main/java/in/koreatech/bus/screen/timetable/viewmodel/BusTimetableViewModel.kt
Show resolved
Hide resolved
| internal val LocalOnRefresh = staticCompositionLocalOf { | ||
| {} | ||
| } |
There was a problem hiding this comment.
이 친구는 용도가 어떤건가요? 어떻세 쓰는지도 궁금해요!
There was a problem hiding this comment.
에러 상황일 경우 위 pr의 사진처럼 고래랑 다시 시도하기 버튼이 나타나는데, 다시 시도하기 버튼 클릭 시 수행할 동작을 지정할 때 사용해용
얘를 안쓰면 onRefresh 매개변수를 계속 전달해줘야 하는데, 그게 귀찮아서 썼어용
CompositionLocalProvider(LocalOnRefresh provides viewModel::refresh) 가 몇 군데에서 쓰이고 있는데, 이 구문으로 감싸진 컴포저블에선 onRefresh 동작으로 viewModel.refresh()를 호출하게 되는거죠!!
There was a problem hiding this comment.
아 이해했어요!! 🤗
이번에 컴포즈 시작하면서 콜백이 늘어나는게 정말 어지러웠는데 ㅋㅋㅋㅋ 한개라도 줄인다는게 너무너무 좋은데요..!! 😁
🐾 개요
🐳 상세 작업
📷 결과 화면
11.mp4
22.mp4
📢 To Reviewers
라인 수가 좀 많네요. 죄송합니다 ㅎㅎ,,,,🙇🙇♂️🙇♀️
그래도 거의 절반 이상이 반복적인 api구조랑 가짜 데이터이니 쫌만 양해부탁드립니다... (그래도 많긴함🙈)
슥슥 넘기면서 봐주세요...ㅎㅎㅎ