fix: dont start a lookup on each key change, throttle a bit. Nicer an…

…d prevent crashes on older devices
farfromrefug · Jan 10, 2025 · d293e45 · d293e45
1 parent d603674
commit d293e45
Showing 1 changed file with 24 additions and 2 deletions.
diff --git a/src/itkach/aard2/lookup/LookupFragment.java b/src/itkach/aard2/lookup/LookupFragment.java
@@ -16,6 +16,9 @@
 import androidx.fragment.app.FragmentActivity;
 import androidx.lifecycle.ViewModelProvider;
 
+import java.util.Timer;
+import java.util.TimerTask;
+
 import itkach.aard2.Application;
 import itkach.aard2.BaseListFragment;
 import itkach.aard2.MainActivity;
@@ -29,6 +32,7 @@
 public class LookupFragment extends BaseListFragment implements LookupListener, SearchView.OnQueryTextListener {
     private final static String TAG = LookupFragment.class.getSimpleName();
 
+    private Timer timer;
     private SearchView searchView;
     private LookupResultAdapter listAdapter;
     private LookupViewModel viewModel;
@@ -95,6 +99,7 @@ public void onPause() {
 
     @Override
     public void onCreateOptionsMenu(@NonNull Menu menu, @NonNull MenuInflater inflater) {
+        timer = new Timer();
         inflater.inflate(R.menu.lookup, menu);
         MenuItem lookupMenu = menu.findItem(R.id.action_lookup);
         View filterActionView = lookupMenu.getActionView();
@@ -173,10 +178,27 @@ public boolean onQueryTextSubmit(String query) {
         return false;
     }
 
+    TimerTask scheduledLookup = null;
     @Override
     public boolean onQueryTextChange(String newText) {
-        if (viewModel != null) {
-            viewModel.lookup(newText);
+//        Log.d(TAG, "new query text: " + newText);
+        TimerTask doLookup = new TimerTask() {
+            @Override
+            public void run() {
+                final String query = searchView.getQuery().toString();
+                if (viewModel != null) {
+                    viewModel.lookup(query);
+                }
+                scheduledLookup = null;
+            }
+        };
+        final String query = searchView.getQuery().toString();
+        if (!AppPrefs.getLastQuery().equals(query)) {
+            if (scheduledLookup != null) {
+                scheduledLookup.cancel();
+            }
+            scheduledLookup = doLookup;
+            timer.schedule(doLookup, 500);
         }
         return true;
     }