Closed luongvo closed 1 year ago
8 Warnings | |
---|---|
:warning: | /home/runner/.gradle/caches/transforms-3/a0b25fd073ab83b080435226956ed8d6/transformed/jetified-dagger-lint-aar-2.43.2/jars/lint.jar#L0 - Lint found an issue registry (dagger.lint.DaggerIssueRegistry ) which requires a newer API level. That means that the custom lint checks are intended for a newer lint version; please upgrade. |
:warning: | app/src/main/AndroidManifest.xml#L10 - On SDK version 23 and up, your app data will be automatically backed up and restored on app install. Consider adding the attribute android:fullBackupContent to specify an @xml resource which configures which files to backup, or just set android:fullBackupOnly=true . More info: https://developer.android.com/guide/topics/data/autobackup |
:warning: | app/src/main/AndroidManifest.xml#L13 - Attribute networkSecurityConfig is only used in API level 24 and higher (current min is 23) |
:warning: | app/src/main/java/co/nimblehq/compose/crypto/ui/common/Toaster.kt#L16 - Toast created but not shown: did you forget to call show() ? |
:warning: | app/src/main/res/drawable/ic_back.xml#L8 - Attribute fillType is only used in API level 24 and higher (current min is 23) |
:warning: | app/src/main/res/drawable/ic_fire_opal_arrow_down.xml#L8 - Attribute fillType is only used in API level 24 and higher (current min is 23) |
:warning: | app/src/main/res/drawable/ic_guppie_green_arrow_up.xml#L8 - Attribute fillType is only used in API level 24 and higher (current min is 23) |
:warning: | app/src/main/res/navigation/nav_graph_main.xml#L2 - The resource R.navigation.nav_graph_main appears to be unused |
Generated by :no_entry_sign: Danger
Resolves https://github.com/nimblehq/jetpack-compose-crypto/issues/32
What happened 👀
load more
feature on the Trending Coin list on the bottom.load more
loading progress at the end of the list.Insight 📝
To build the load more feature, we need to set
rememberLazyListState
to the LazyColumn's state. Then, when rendering list items, we will check to current render index to thetrendingCoins.lastIndex
to trigger the load more action in ViewModel.After implementing the load more function, the current loading progress will replace the current Trending list page items. To fix this, we need to show the load more's loading progress at the end of the list instead of replacing the whole list item. The need is to create a new sealed class
LoadingState
with 3 states Idle (~ false), Loading (~ true) and LoadingMore (the newtrue
with load more).Proof Of Work 📹
https://user-images.githubusercontent.com/16315358/200374368-0a6d3456-8dca-4c10-9d4d-b84180f75eb0.mp4