From 4688da7c04ba3009b71aef297a65c6338ed39880 Mon Sep 17 00:00:00 2001
From: HapeLee <1321903405@qq.com>
Date: Mon, 27 Oct 2025 01:45:30 +0800
Subject: [PATCH] =?UTF-8?q?=E4=BC=98=E5=8C=96=E6=90=9C=E7=B4=A2=E9=A1=B5?=
 =?UTF-8?q?=E9=9D=A2=E7=9A=84=E6=8C=89=E9=92=AE=E6=96=87=E5=AD=97=E6=98=BE?=
 =?UTF-8?q?=E7=A4=BA=20#149?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 .../kazusa/ui/book/search/SearchActivity.kt   | 56 +++++++++++++------
 .../main/res/layout/activity_book_search.xml  |  7 ++-
 app/version.properties                        |  2 +-
 3 files changed, 45 insertions(+), 20 deletions(-)

diff --git a/app/src/main/java/io/legato/kazusa/ui/book/search/SearchActivity.kt b/app/src/main/java/io/legato/kazusa/ui/book/search/SearchActivity.kt
index 181978e09..284ecd83b 100644
--- a/app/src/main/java/io/legato/kazusa/ui/book/search/SearchActivity.kt
+++ b/app/src/main/java/io/legato/kazusa/ui/book/search/SearchActivity.kt
@@ -114,6 +114,7 @@ class SearchActivity : VMBaseActivity<ActivityBookSearchBinding, SearchViewModel
 
     override fun onMenuOpened(featureId: Int, menu: Menu): Boolean {
         buildSearchScopeMenu(menu)
+        updateSearchScopeChip()
         return super.onMenuOpened(featureId, menu)
     }
 
@@ -161,36 +162,43 @@ class SearchActivity : VMBaseActivity<ActivityBookSearchBinding, SearchViewModel
             alertSearchScope()
         }
         binding.chipGroup.setOnClickListener { view ->
+            binding.chipGroup.isChecked = true
             val popup = PopupMenu(this, view)
             val menu = popup.menu
             buildSearchScopeMenu(menu)
+            updateSearchScopeChip()
+            var fromGroup1 = false
             popup.setOnMenuItemClickListener { item ->
                 when (item.groupId) {
-                    R.id.menu_group_1 -> item.title.toString()
+                    R.id.menu_group_1 -> {
+                        viewModel.searchScope.remove(item.title.toString())
+                        fromGroup1 = true
+                    }
                     R.id.menu_group_2 -> {
                         if (item.itemId == R.id.menu_1) {
-                            ""
+                            viewModel.searchScope.update("")
+                            fromGroup1 = true
                         } else {
-                            item.title.toString()
+                            viewModel.searchScope.update(item.title.toString())
+                            fromGroup1 = false
                         }
                     }
-                    else -> item.title.toString()
-                }
-                if (item.groupId == R.id.menu_group_1) {
-                    viewModel.searchScope.remove(item.title.toString())
-                } else {
-                    if (item.itemId == R.id.menu_1) {
-                        viewModel.searchScope.update("")
-                    } else {
+                    else -> {
                         viewModel.searchScope.update(item.title.toString())
+                        fromGroup1 = false
                     }
                 }
+
                 updateSelectedGroup()
                 true
             }
-
+            popup.setOnDismissListener {
+                binding.chipGroup.isChecked = !fromGroup1
+                updateSearchScopeChip()
+            }
             popup.show()
         }
+
         searchBar.setOnMenuItemClickListener { item ->
             onCompatOptionsItemSelected(item)
         }
@@ -257,12 +265,10 @@ class SearchActivity : VMBaseActivity<ActivityBookSearchBinding, SearchViewModel
     }
 
     private fun updateSelectedGroup() {
-        val chip = binding.chipGroup
         val displayNames = viewModel.searchScope.displayNames
         val groupName = displayNames.firstOrNull().orEmpty()
-        chip.text = groupName.ifEmpty { getString(R.string.search_select_group) }
-        chip.isCheckable = true
-        chip.isChecked = groupName.isNotEmpty()
+        binding.chipGroup.text = groupName.ifEmpty { getString(R.string.search_select_group) }
+        binding.chipGroup.isChecked = groupName.isNotEmpty()
     }
 
     private fun buildSearchScopeMenu(menu: Menu) {
@@ -609,6 +615,24 @@ class SearchActivity : VMBaseActivity<ActivityBookSearchBinding, SearchViewModel
 
     override fun onSearchScopeOk(searchScope: SearchScope) {
         viewModel.searchScope.update(searchScope.toString())
+        updateSearchScopeChip()
+    }
+
+    private fun updateSearchScopeChip() {
+        binding.chipSearchScope.isCheckable = true
+        val scopeNames = viewModel.searchScope.displayNames
+        if (scopeNames.isEmpty() || viewModel.searchScope.isAll()) {
+            binding.chipSearchScope.apply {
+                isChecked = false
+                text = getString(R.string.all_source)
+            }
+        } else {
+            binding.chipSearchScope.apply {
+                isChecked = true
+                text = scopeNames.joinToString(", ")
+            }
+        }
+        binding.chipSearchScope.isCheckable = false
     }
 
     private fun alertSearchScope() {
diff --git a/app/src/main/res/layout/activity_book_search.xml b/app/src/main/res/layout/activity_book_search.xml
index d685ebf78..4dc3d7aac 100644
--- a/app/src/main/res/layout/activity_book_search.xml
+++ b/app/src/main/res/layout/activity_book_search.xml
@@ -127,6 +127,7 @@
             <com.google.android.material.chip.ChipGroup
                 android:layout_width="match_parent"
                 android:layout_height="wrap_content"
+                app:chipSpacingVertical="0dp"
                 android:layout_marginTop="8dp"
                 android:layout_marginHorizontal="16dp">
                 <com.google.android.material.chip.Chip
@@ -139,13 +140,13 @@
                     android:id="@+id/chip_search_scope"
                     android:layout_width="wrap_content"
                     android:layout_height="wrap_content"
-                    android:clickable="true"
-                    android:text="@string/search_scope"/>
+                    android:text="@string/all_source"/>
                 <com.google.android.material.chip.Chip
                     android:id="@+id/chip_group"
                     android:layout_width="wrap_content"
                     android:layout_height="wrap_content"
-                    android:clickable="true"
+                    android:visibility="gone"
+                    android:checkable="true"
                     android:text="@string/search_select_group"/>
             </com.google.android.material.chip.ChipGroup>
 
diff --git a/app/version.properties b/app/version.properties
index 15005a673..2144a2482 100644
--- a/app/version.properties
+++ b/app/version.properties
@@ -1,4 +1,4 @@
 VERSION_MAJOR=3
 VERSION_MINOR=25
-VERSION_PATCH=12
+VERSION_PATCH=13
 VERSION_SUFFIX=
\ No newline at end of file