Merge pull request #4117 from kiwix/Fixes#4112

Fixed: Virtual keyboard reset the search (in library).

app/src/main/java/org/kiwix/kiwixmobile/nav/destination/library/OnlineLibraryFragment.kt

@@ -68,6 +68,7 @@ import org.kiwix.kiwixmobile.core.extensions.ActivityExtensions.requestNotificat
 import org.kiwix.kiwixmobile.core.extensions.ActivityExtensions.viewModel
 import org.kiwix.kiwixmobile.core.extensions.closeKeyboard
 import org.kiwix.kiwixmobile.core.extensions.coreMainActivity
+import org.kiwix.kiwixmobile.core.extensions.isKeyboardVisible
 import org.kiwix.kiwixmobile.core.extensions.setBottomMarginToFragmentContainerView
 import org.kiwix.kiwixmobile.core.extensions.setUpSearchView
 import org.kiwix.kiwixmobile.core.extensions.snack
@@ -327,9 +328,12 @@ class OnlineLibraryFragment : BaseFragment(), FragmentActivityExtensions {
   }
 
   override fun onBackPressed(activity: AppCompatActivity): FragmentActivityExtensions.Super {
-    getActivity()?.finish()
+    if (isKeyboardVisible()) {
+      closeKeyboard()
       return FragmentActivityExtensions.Super.ShouldNotCall
     }
+    return FragmentActivityExtensions.Super.ShouldCall
+  }
 
   private fun onRefreshStateChange(isRefreshing: Boolean?) {
     var refreshing = isRefreshing == true

core/src/main/AndroidManifest.xml

@@ -55,7 +55,6 @@
     android:fullBackupContent="@xml/backup_rules"
     android:dataExtractionRules = "@xml/data_extraction_rules"
     android:hardwareAccelerated="true"
-    android:enableOnBackInvokedCallback="true"
     android:largeHeap="true"
     android:requestLegacyExternalStorage="true"
     android:resizeableActivity="true"

core/src/main/java/org/kiwix/kiwixmobile/core/extensions/FragmentExtensions.kt

@@ -22,6 +22,8 @@ import android.content.Context
 import android.view.View
 import android.view.inputmethod.InputMethodManager
 import android.widget.Toast
+import androidx.core.view.ViewCompat
+import androidx.core.view.WindowInsetsCompat
 import androidx.fragment.app.Fragment
 import androidx.lifecycle.ViewModel
 import androidx.lifecycle.ViewModelProvider
@@ -36,6 +38,11 @@ fun Fragment.toast(stringId: Int, length: Int = Toast.LENGTH_LONG) {
   requireActivity().toast(stringId, length)
 }
 
+fun Fragment.isKeyboardVisible(): Boolean {
+  val insets = ViewCompat.getRootWindowInsets(requireView()) ?: return false
+  return insets.isVisible(WindowInsetsCompat.Type.ime())
+}
+
 fun Fragment.closeKeyboard() {
   val inputMethodManager =
     requireContext().getSystemService(Context.INPUT_METHOD_SERVICE) as InputMethodManager

core/src/main/java/org/kiwix/kiwixmobile/core/main/CoreMainActivity.kt

@@ -24,6 +24,7 @@ import android.os.Process
 import android.view.ActionMode
 import android.view.Menu
 import android.view.MenuItem
+import androidx.activity.OnBackPressedCallback
 import androidx.appcompat.app.ActionBarDrawerToggle
 import androidx.appcompat.app.AppCompatActivity
 import androidx.appcompat.widget.Toolbar
@@ -47,6 +48,7 @@ import org.kiwix.kiwixmobile.core.CoreApp
 import org.kiwix.kiwixmobile.core.R
 import org.kiwix.kiwixmobile.core.base.BaseActivity
 import org.kiwix.kiwixmobile.core.base.FragmentActivityExtensions
+import org.kiwix.kiwixmobile.core.base.FragmentActivityExtensions.Super.ShouldCall
 import org.kiwix.kiwixmobile.core.data.remote.ObjectBoxToLibkiwixMigrator
 import org.kiwix.kiwixmobile.core.data.remote.ObjectBoxToRoomMigrator
 import org.kiwix.kiwixmobile.core.di.components.CoreActivityComponent
@@ -132,6 +134,7 @@ abstract class CoreMainActivity : BaseActivity(), WebViewProvider {
     }
     downloadManagerBroadcastReceiver.let(::registerReceiver)
     createApplicationShortcuts()
+    handleBackPressed()
   }
 
   @Suppress("DEPRECATION")
@@ -149,6 +152,7 @@ abstract class CoreMainActivity : BaseActivity(), WebViewProvider {
   }
 
   override fun onDestroy() {
+    onBackPressedCallBack.remove()
     downloadManagerBroadcastReceiver.let(::unregisterReceiver)
     super.onDestroy()
   }
@@ -277,16 +281,25 @@ abstract class CoreMainActivity : BaseActivity(), WebViewProvider {
     externalLinkOpener.openExternalUrl(KIWIX_SUPPORT_URL.toUri().browserIntent())
   }
 
-  override fun onBackPressed() {
+  private fun handleBackPressed() {
+    onBackPressedDispatcher.addCallback(this, onBackPressedCallBack)
+  }
+
+  private val onBackPressedCallBack =
+    object : OnBackPressedCallback(true) {
+      override fun handleOnBackPressed() {
         if (navigationDrawerIsOpen()) {
           closeNavigationDrawer()
           return
         }
         if (activeFragments().filterIsInstance<FragmentActivityExtensions>().isEmpty()) {
-      return super.onBackPressedDispatcher.onBackPressed()
+          isEnabled = false
+          return onBackPressedDispatcher.onBackPressed().also {
+            isEnabled = true
+          }
         }
         activeFragments().filterIsInstance<FragmentActivityExtensions>().forEach {
-      if (it.onBackPressed(this) == FragmentActivityExtensions.Super.ShouldCall) {
+          if (it.onBackPressed(this@CoreMainActivity) == ShouldCall) {
             if (navController.currentDestination?.id?.equals(readerFragmentResId) == true &&
               navController.previousBackStackEntry?.destination
                 ?.id?.equals(searchFragmentResId) == false
@@ -294,7 +307,10 @@ abstract class CoreMainActivity : BaseActivity(), WebViewProvider {
               drawerToggle = null
               finish()
             } else {
-          super.onBackPressedDispatcher.onBackPressed()
+              isEnabled = false
+              onBackPressedDispatcher.onBackPressed()
+              isEnabled = true
+            }
           }
         }
       }