fix for searchview anormal behavior

libre-tube · Sep 6, 2022 · ca3734a · ca3734a
1 parent f66fdf1
commit ca3734a
Showing 1 changed file with 2 additions and 15 deletions.
diff --git a/app/src/main/java/com/github/libretube/activities/MainActivity.kt b/app/src/main/java/com/github/libretube/activities/MainActivity.kt
@@ -162,8 +162,6 @@ class MainActivity : BaseActivity() {
         // new way of handling back presses
         onBackPressedDispatcher.addCallback(object : OnBackPressedCallback(true) {
             override fun handleOnBackPressed() {
-                // remove focus from search
-                removeSearchFocus()
                 navController.popBackStack(R.id.searchFragment, false)
 
                 if (binding.mainMotionLayout.progress == 0F) {
@@ -260,6 +258,7 @@ class MainActivity : BaseActivity() {
                 val bundle = Bundle()
                 bundle.putString("query", query)
                 navController.navigate(R.id.searchResultFragment, bundle)
+                searchViewModel.setQuery("")
                 return true
             }
 
@@ -284,24 +283,12 @@ class MainActivity : BaseActivity() {
                 override fun onMenuItemActionCollapse(p0: MenuItem): Boolean {
                     val currentFragmentId = navController.currentDestination?.id
                     if (currentFragmentId == R.id.searchFragment || currentFragmentId == R.id.searchResultFragment) {
-                        try {
-                            onBackPressedDispatcher.onBackPressed()
-                        } catch (e: Exception) {
-                            e.printStackTrace()
-                        }
+                        navController.popBackStack()
                     }
                     return true
                 }
             }
         )
-
-        searchView.setOnCloseListener {
-            if (navController.currentDestination?.id == R.id.searchFragment) {
-                searchViewModel.setQuery(null)
-                onBackPressedDispatcher.onBackPressed()
-            }
-            false
-        }
         return super.onCreateOptionsMenu(menu)
     }