fix: can't update home page when request failed

PullToRefreshBox relies on scroll state of child composable.
Make child composable has full size to fix it.

Author: heddxh

app/src/main/kotlin/day/vitayuzu/neodb/ui/page/home/HomeScreen.kt

@@ -7,7 +7,7 @@ import androidx.compose.foundation.layout.Column
 import androidx.compose.foundation.layout.PaddingValues
 import androidx.compose.foundation.layout.Spacer
 import androidx.compose.foundation.layout.consumeWindowInsets
-import androidx.compose.foundation.layout.fillMaxWidth
+import androidx.compose.foundation.layout.fillMaxSize
 import androidx.compose.foundation.layout.height
 import androidx.compose.foundation.layout.padding
 import androidx.compose.foundation.pager.HorizontalPager
@@ -53,28 +53,27 @@ fun HomeScreen(modifier: Modifier = Modifier, viewModel: HomeViewModel = hiltVie
     val appNavigator = LocalNavigator.current
 
     // TODO: show fetched data immediately
-    PullToRefreshBox(
-        isRefreshing = uiState.isLoading,
-        onRefresh = { viewModel.updateTrending() },
+    Scaffold(
         modifier = modifier,
+        bottomBar = { SharedBottomBar() },
+        floatingActionButton = {
+            SharedFab(onClick = { appNavigator goto AppNavigator.Search }) {
+                Icon(
+                    imageVector = Icons.Default.Search,
+                    contentDescription = "Perform search",
+                )
+            }
+        },
     ) {
-        Scaffold(
-            bottomBar = { SharedBottomBar() },
-            floatingActionButton = {
-                SharedFab(onClick = { appNavigator goto AppNavigator.Search }) {
-                    Icon(
-                        imageVector = Icons.Default.Search,
-                        contentDescription = "Perform search",
-                    )
-                }
-            },
+        PullToRefreshBox(
+            isRefreshing = uiState.isLoading,
+            onRefresh = { viewModel.updateTrending() },
+            modifier = Modifier.padding(it).consumeWindowInsets(it),
         ) {
             Column(
-                modifier = Modifier
-                    .padding(it)
-                    .consumeWindowInsets(it)
-                    .fillMaxWidth()
-                    .verticalScroll(rememberScrollState()),
+                // PullToRefresh relies on child scroll event to detect scroll gestures,
+                // so make sure child composable has enough height.
+                modifier = Modifier.fillMaxSize().verticalScroll(rememberScrollState()),
                 verticalArrangement = Arrangement.spacedBy(16.dp, Alignment.Top),
             ) {
                 // Book

app/src/main/kotlin/day/vitayuzu/neodb/ui/page/library/LibraryPage.kt

@@ -4,6 +4,7 @@ import androidx.compose.foundation.background
 import androidx.compose.foundation.layout.Arrangement
 import androidx.compose.foundation.layout.PaddingValues
 import androidx.compose.foundation.layout.consumeWindowInsets
+import androidx.compose.foundation.layout.fillMaxSize
 import androidx.compose.foundation.layout.padding
 import androidx.compose.foundation.lazy.LazyColumn
 import androidx.compose.foundation.lazy.LazyRow
@@ -37,22 +38,25 @@ import day.vitayuzu.neodb.util.ShelfType
 fun LibraryPage(modifier: Modifier = Modifier, viewModel: LibraryViewModel = hiltViewModel()) {
     val uiState by viewModel.uiState.collectAsStateWithLifecycle()
 
-    PullToRefreshBox(
-        isRefreshing = uiState.isLoading,
-        onRefresh = viewModel::refresh,
+    val appNavigator = LocalNavigator.current
+    Scaffold(
         modifier = modifier,
-    ) {
-        val appNavigator = LocalNavigator.current
-
-        Scaffold(bottomBar = { SharedBottomBar() }, floatingActionButton = {
+        bottomBar = { SharedBottomBar() },
+        floatingActionButton = {
             SharedFab(onClick = { appNavigator goto AppNavigator.Search }) {
                 Icon(
                     imageVector = Icons.Default.Search,
                     contentDescription = "Perform search",
                 )
             }
-        }) { paddings ->
-            LazyColumn(Modifier.padding(paddings).consumeWindowInsets(paddings)) {
+        },
+    ) {
+        PullToRefreshBox(
+            isRefreshing = uiState.isLoading,
+            onRefresh = viewModel::refresh,
+            modifier = Modifier.padding(it).consumeWindowInsets(it),
+        ) {
+            LazyColumn(Modifier.fillMaxSize()) {
                 stickyHeader {
                     // FIXME: In some locale like Chinese we don't need scrollable row
                     //  so it is not centered.