regression: scroll mode disabled if paging button hidden since 5d5f7d5

Author: eagleoflqj

.github/workflows/ci.yml

@@ -13,7 +13,7 @@ jobs:
   lint:
     runs-on: macos-26
     steps:
-      - uses: actions/checkout@v5
+      - uses: actions/checkout@v6
         with:
           submodules: true # needs fcitx5/.clang-format
 
@@ -36,11 +36,11 @@ jobs:
           - { arch: x86_64, type: Debug, postfix: -debug }
 
     steps:
-      - uses: actions/checkout@v5
+      - uses: actions/checkout@v6
         with:
           submodules: recursive
 
-      - uses: actions/setup-node@v5
+      - uses: actions/setup-node@v6
         with:
           node-version: 24.x
 

webpanel/webpanel.cpp

@@ -426,10 +426,9 @@ void WebPanel::update(UserInterfaceComponent component,
             }
             // Paging
             auto *pageableList = list->toPageable();
-            pageable =
-                pageableList && *config_.typography->pagingButtonsStyle !=
-                                    PagingButtonsStyle::None;
-            if (pageable) {
+            if (pageableList) {
+                pageable = *config_.typography->pagingButtonsStyle !=
+                           PagingButtonsStyle::None;
                 hasPrev = pageableList->hasPrev();
                 hasNext = pageableList->hasNext();
             }
@@ -448,7 +447,7 @@ void WebPanel::update(UserInterfaceComponent component,
                     return expand();
                 }
                 // Disable scroll mode if all candidates are on the same page.
-                if (pageableList != nullptr && (hasPrev || hasNext)) {
+                if (hasPrev || hasNext) {
                     scrollState_ = candidate_window::scroll_state_t::ready;
                 } else {
                     pageable = false;