Fix topic scrollbar item count for news loading and error states

Account for TopicToolbar and TopicHeader when computing itemsAvailable for TopicUiState.Success.
4 months ago · 67cdbe8c3d
parent 4ed499289a
commit 67cdbe8c3d
1 changed files with 3 additions and 3 deletions
--- a/feature/topic/impl/src/main/kotlin/com/google/samples/apps/nowinandroid/feature/topic/impl/TopicScreen.kt
+++ b/feature/topic/impl/src/main/kotlin/com/google/samples/apps/nowinandroid/feature/topic/impl/TopicScreen.kt
@ -190,9 +190,9 @@ private fun topicItemsSize(
    TopicUiState.Error -> 2 // Toolbar and error message
    TopicUiState.Loading -> 1 // Loading bar
    is TopicUiState.Success -> when (newsUiState) {
-        NewsUiState.Error -> 0 // Nothing
+        NewsUiState.Error -> 3 // Toolbar, header, and error message
-        NewsUiState.Loading -> 1 // Loading bar
+        NewsUiState.Loading -> 3 // Toolbar, header, and loading indicator
-        is NewsUiState.Success -> 2 + newsUiState.news.size // Toolbar, header
+        is NewsUiState.Success -> 2 + newsUiState.news.size // Toolbar, header, and news items
    }
 }