Search improvements

pull/153/head
M66B 7 years ago
parent b20f62f577
commit 0d330fc481

@ -24,7 +24,11 @@ import android.animation.ValueAnimator;
import android.content.Context; import android.content.Context;
import android.content.Intent; import android.content.Intent;
import android.os.Bundle; import android.os.Bundle;
import android.text.TextUtils;
import android.view.LayoutInflater; import android.view.LayoutInflater;
import android.view.Menu;
import android.view.MenuInflater;
import android.view.MenuItem;
import android.view.View; import android.view.View;
import android.view.ViewGroup; import android.view.ViewGroup;
@ -35,6 +39,7 @@ import java.util.List;
import androidx.annotation.NonNull; import androidx.annotation.NonNull;
import androidx.annotation.Nullable; import androidx.annotation.Nullable;
import androidx.appcompat.widget.SearchView;
import androidx.constraintlayout.widget.Group; import androidx.constraintlayout.widget.Group;
import androidx.fragment.app.FragmentTransaction; import androidx.fragment.app.FragmentTransaction;
import androidx.lifecycle.Observer; import androidx.lifecycle.Observer;
@ -51,6 +56,7 @@ public class FragmentAccounts extends FragmentBase {
private FloatingActionButton fabCompose; private FloatingActionButton fabCompose;
private ObjectAnimator animator; private ObjectAnimator animator;
private String searching = null;
private AdapterAccount adapter; private AdapterAccount adapter;
@Override @Override
@ -64,6 +70,7 @@ public class FragmentAccounts extends FragmentBase {
@Nullable @Nullable
public View onCreateView(@NonNull LayoutInflater inflater, @Nullable ViewGroup container, @Nullable Bundle savedInstanceState) { public View onCreateView(@NonNull LayoutInflater inflater, @Nullable ViewGroup container, @Nullable Bundle savedInstanceState) {
setSubtitle(R.string.title_list_accounts); setSubtitle(R.string.title_list_accounts);
setHasOptionsMenu(true);
View view = inflater.inflate(R.layout.fragment_accounts, container, false); View view = inflater.inflate(R.layout.fragment_accounts, container, false);
@ -155,10 +162,20 @@ public class FragmentAccounts extends FragmentBase {
return view; return view;
} }
@Override
public void onSaveInstanceState(Bundle outState) {
super.onSaveInstanceState(outState);
outState.putString("fair:searching", searching);
}
@Override @Override
public void onActivityCreated(@Nullable Bundle savedInstanceState) { public void onActivityCreated(@Nullable Bundle savedInstanceState) {
super.onActivityCreated(savedInstanceState); super.onActivityCreated(savedInstanceState);
if (savedInstanceState != null)
searching = savedInstanceState.getString("fair:searching");
DB db = DB.getInstance(getContext()); DB db = DB.getInstance(getContext());
// Observe accounts // Observe accounts
@ -194,4 +211,43 @@ public class FragmentAccounts extends FragmentBase {
} }
}); });
} }
@Override
public void onCreateOptionsMenu(Menu menu, MenuInflater inflater) {
inflater.inflate(R.menu.menu_folders, menu);
final MenuItem menuSearch = menu.findItem(R.id.menu_search);
SearchView searchView = (SearchView) menuSearch.getActionView();
if (!TextUtils.isEmpty(searching)) {
menuSearch.expandActionView();
searchView.setQuery(searching, false);
}
searchView.setQueryHint(getString(R.string.title_search_device));
searchView.setOnQueryTextListener(new SearchView.OnQueryTextListener() {
@Override
public boolean onQueryTextChange(String newText) {
searching = newText;
return true;
}
@Override
public boolean onQueryTextSubmit(String query) {
searching = null;
menuSearch.collapseActionView();
FragmentMessages.search(getContext(), getViewLifecycleOwner(), getFragmentManager(), -1, query);
return true;
}
});
super.onCreateOptionsMenu(menu, inflater);
}
@Override
public void onPrepareOptionsMenu(Menu menu) {
menu.findItem(R.id.menu_search).setVisible(!settings);
super.onPrepareOptionsMenu(menu);
}
} }

@ -50,7 +50,7 @@ public class FragmentContacts extends FragmentBase {
private ContentLoadingProgressBar pbWait; private ContentLoadingProgressBar pbWait;
private Group grpReady; private Group grpReady;
private String search = null; private String searching = null;
private AdapterContact adapter; private AdapterContact adapter;
@Override @Override
@ -82,13 +82,19 @@ public class FragmentContacts extends FragmentBase {
return view; return view;
} }
@Override
public void onSaveInstanceState(Bundle outState) {
super.onSaveInstanceState(outState);
outState.putString("fair:searching", searching);
}
@Override @Override
public void onActivityCreated(@Nullable Bundle savedInstanceState) { public void onActivityCreated(@Nullable Bundle savedInstanceState) {
super.onActivityCreated(savedInstanceState); super.onActivityCreated(savedInstanceState);
if (savedInstanceState != null) { if (savedInstanceState != null) {
search = savedInstanceState.getString("fair:search"); searching = savedInstanceState.getString("fair:searching");
adapter.search(search); adapter.search(searching);
} }
DB db = DB.getInstance(getContext()); DB db = DB.getInstance(getContext());
@ -108,12 +114,6 @@ public class FragmentContacts extends FragmentBase {
Shortcuts.update(getContext(), getViewLifecycleOwner()); Shortcuts.update(getContext(), getViewLifecycleOwner());
} }
@Override
public void onSaveInstanceState(Bundle outState) {
super.onSaveInstanceState(outState);
outState.putString("fair:search", search);
}
@Override @Override
public void onCreateOptionsMenu(Menu menu, MenuInflater inflater) { public void onCreateOptionsMenu(Menu menu, MenuInflater inflater) {
inflater.inflate(R.menu.menu_contacts, menu); inflater.inflate(R.menu.menu_contacts, menu);
@ -121,24 +121,23 @@ public class FragmentContacts extends FragmentBase {
MenuItem menuSearch = menu.findItem(R.id.menu_search); MenuItem menuSearch = menu.findItem(R.id.menu_search);
SearchView searchView = (SearchView) menuSearch.getActionView(); SearchView searchView = (SearchView) menuSearch.getActionView();
if (!TextUtils.isEmpty(search)) { if (!TextUtils.isEmpty(searching)) {
menuSearch.expandActionView(); menuSearch.expandActionView();
searchView.setQuery(search, true); searchView.setQuery(searching, true);
searchView.clearFocus();
} }
searchView.setOnQueryTextListener(new SearchView.OnQueryTextListener() { searchView.setOnQueryTextListener(new SearchView.OnQueryTextListener() {
@Override @Override
public boolean onQueryTextSubmit(String query) { public boolean onQueryTextChange(String newText) {
search = query; searching = newText;
adapter.search(query); adapter.search(newText);
return true; return true;
} }
@Override @Override
public boolean onQueryTextChange(String newText) { public boolean onQueryTextSubmit(String query) {
search = newText; searching = query;
adapter.search(newText); adapter.search(query);
return true; return true;
} }
}); });

@ -24,6 +24,7 @@ import android.content.Intent;
import android.content.SharedPreferences; import android.content.SharedPreferences;
import android.graphics.Color; import android.graphics.Color;
import android.os.Bundle; import android.os.Bundle;
import android.text.TextUtils;
import android.view.LayoutInflater; import android.view.LayoutInflater;
import android.view.Menu; import android.view.Menu;
import android.view.MenuInflater; import android.view.MenuInflater;
@ -65,7 +66,7 @@ public class FragmentFolders extends FragmentBase {
private FloatingActionButton fab; private FloatingActionButton fab;
private long account; private long account;
private boolean searching = false; private String searching = null;
private AdapterFolder adapter; private AdapterFolder adapter;
private Map<Long, List<TupleFolderEx>> parentChilds = new HashMap<>(); private Map<Long, List<TupleFolderEx>> parentChilds = new HashMap<>();
@ -79,34 +80,11 @@ public class FragmentFolders extends FragmentBase {
account = args.getLong("account", -1); account = args.getLong("account", -1);
} }
@Override
public void onSaveInstanceState(Bundle outState) {
super.onSaveInstanceState(outState);
outState.putLongArray("fair:parents", Helper.toLongArray(parentChilds.keySet()));
for (Long parent : parentChilds.keySet()) {
List<TupleFolderEx> childs = parentChilds.get(parent);
outState.putInt("fair:childs:" + parent + ":count", childs.size());
for (int i = 0; i < childs.size(); i++)
outState.putSerializable("fair:childs:" + parent + ":" + i, childs.get(i));
}
}
@Override @Override
@Nullable @Nullable
public View onCreateView(@NonNull LayoutInflater inflater, @Nullable ViewGroup container, @Nullable Bundle savedInstanceState) { public View onCreateView(@NonNull LayoutInflater inflater, @Nullable ViewGroup container, @Nullable Bundle savedInstanceState) {
setHasOptionsMenu(true); setHasOptionsMenu(true);
if (savedInstanceState != null) {
for (long parent : savedInstanceState.getLongArray("fair:parents")) {
int count = savedInstanceState.getInt("fair:childs:" + parent + ":count");
List<TupleFolderEx> childs = new ArrayList<>(count);
for (int i = 0; i < count; i++)
childs.add((TupleFolderEx) savedInstanceState.getSerializable("fair:childs:" + parent + ":" + i));
parentChilds.put(parent, childs);
}
}
view = (ViewGroup) inflater.inflate(R.layout.fragment_folders, container, false); view = (ViewGroup) inflater.inflate(R.layout.fragment_folders, container, false);
// Get controls // Get controls
@ -225,10 +203,37 @@ public class FragmentFolders extends FragmentBase {
return view; return view;
} }
@Override
public void onSaveInstanceState(Bundle outState) {
super.onSaveInstanceState(outState);
outState.putString("fair:searching", searching);
outState.putLongArray("fair:parents", Helper.toLongArray(parentChilds.keySet()));
for (Long parent : parentChilds.keySet()) {
List<TupleFolderEx> childs = parentChilds.get(parent);
outState.putInt("fair:childs:" + parent + ":count", childs.size());
for (int i = 0; i < childs.size(); i++)
outState.putSerializable("fair:childs:" + parent + ":" + i, childs.get(i));
}
}
@Override @Override
public void onActivityCreated(@Nullable Bundle savedInstanceState) { public void onActivityCreated(@Nullable Bundle savedInstanceState) {
super.onActivityCreated(savedInstanceState); super.onActivityCreated(savedInstanceState);
if (savedInstanceState != null) {
searching = savedInstanceState.getString("fair:searching");
for (long parent : savedInstanceState.getLongArray("fair:parents")) {
int count = savedInstanceState.getInt("fair:childs:" + parent + ":count");
List<TupleFolderEx> childs = new ArrayList<>(count);
for (int i = 0; i < count; i++)
childs.add((TupleFolderEx) savedInstanceState.getSerializable("fair:childs:" + parent + ":" + i));
parentChilds.put(parent, childs);
}
}
SharedPreferences prefs = PreferenceManager.getDefaultSharedPreferences(getContext()); SharedPreferences prefs = PreferenceManager.getDefaultSharedPreferences(getContext());
grpHintActions.setVisibility(prefs.getBoolean("folder_actions", false) ? View.GONE : View.VISIBLE); grpHintActions.setVisibility(prefs.getBoolean("folder_actions", false) ? View.GONE : View.VISIBLE);
grpHintSync.setVisibility(prefs.getBoolean("folder_sync", false) ? View.GONE : View.VISIBLE); grpHintSync.setVisibility(prefs.getBoolean("folder_sync", false) ? View.GONE : View.VISIBLE);
@ -339,37 +344,28 @@ public class FragmentFolders extends FragmentBase {
inflater.inflate(R.menu.menu_folders, menu); inflater.inflate(R.menu.menu_folders, menu);
final MenuItem menuSearch = menu.findItem(R.id.menu_search); final MenuItem menuSearch = menu.findItem(R.id.menu_search);
menuSearch.setOnActionExpandListener(new MenuItem.OnActionExpandListener() { SearchView searchView = (SearchView) menuSearch.getActionView();
@Override
public boolean onMenuItemActionExpand(MenuItem item) { if (!TextUtils.isEmpty(searching)) {
searching = true; menuSearch.expandActionView();
return true; searchView.setQuery(searching, false);
} }
searchView.setQueryHint(getString(R.string.title_search_device));
searchView.setOnQueryTextListener(new SearchView.OnQueryTextListener() {
@Override @Override
public boolean onMenuItemActionCollapse(MenuItem item) { public boolean onQueryTextChange(String newText) {
searching = false; searching = newText;
return true; return true;
} }
});
if (searching)
menuSearch.expandActionView();
final SearchView searchView = (SearchView) menuSearch.getActionView();
searchView.setQueryHint(getString(R.string.title_search_device));
searchView.setOnQueryTextListener(new SearchView.OnQueryTextListener() {
@Override @Override
public boolean onQueryTextSubmit(String query) { public boolean onQueryTextSubmit(String query) {
searching = false; searching = null;
menuSearch.collapseActionView(); menuSearch.collapseActionView();
FragmentMessages.search(getContext(), getViewLifecycleOwner(), getFragmentManager(), -1, query); FragmentMessages.search(getContext(), getViewLifecycleOwner(), getFragmentManager(), -1, query);
return true; return true;
} }
@Override
public boolean onQueryTextChange(String newText) {
return false;
}
}); });
super.onCreateOptionsMenu(menu, inflater); super.onCreateOptionsMenu(menu, inflater);

@ -134,7 +134,7 @@ public class FragmentMessages extends FragmentBase {
private long primary; private long primary;
private boolean outbox = false; private boolean outbox = false;
private boolean connected; private boolean connected;
private boolean searching = false; private String searching = null;
private boolean refresh = false; private boolean refresh = false;
private boolean manual = false; private boolean manual = false;
private AdapterMessage adapter; private AdapterMessage adapter;
@ -1486,6 +1486,8 @@ public class FragmentMessages extends FragmentBase {
public void onSaveInstanceState(Bundle outState) { public void onSaveInstanceState(Bundle outState) {
super.onSaveInstanceState(outState); super.onSaveInstanceState(outState);
outState.putString("fair:searching", searching);
outState.putBoolean("fair:autoExpanded", autoExpanded); outState.putBoolean("fair:autoExpanded", autoExpanded);
outState.putInt("fair:autoCloseCount", autoCloseCount); outState.putInt("fair:autoCloseCount", autoCloseCount);
@ -1515,6 +1517,8 @@ public class FragmentMessages extends FragmentBase {
super.onActivityCreated(savedInstanceState); super.onActivityCreated(savedInstanceState);
if (savedInstanceState != null) { if (savedInstanceState != null) {
searching = savedInstanceState.getString("fair:searching");
autoExpanded = savedInstanceState.getBoolean("fair:autoExpanded"); autoExpanded = savedInstanceState.getBoolean("fair:autoExpanded");
autoCloseCount = savedInstanceState.getInt("fair:autoCloseCount"); autoCloseCount = savedInstanceState.getInt("fair:autoCloseCount");
@ -1765,37 +1769,28 @@ public class FragmentMessages extends FragmentBase {
inflater.inflate(R.menu.menu_messages, menu); inflater.inflate(R.menu.menu_messages, menu);
final MenuItem menuSearch = menu.findItem(R.id.menu_search); final MenuItem menuSearch = menu.findItem(R.id.menu_search);
menuSearch.setOnActionExpandListener(new MenuItem.OnActionExpandListener() { SearchView searchView = (SearchView) menuSearch.getActionView();
@Override
public boolean onMenuItemActionExpand(MenuItem item) { if (!TextUtils.isEmpty(searching)) {
searching = true; menuSearch.expandActionView();
return true; searchView.setQuery(searching, false);
} }
searchView.setQueryHint(getString(folder < 0 ? R.string.title_search_device : R.string.title_search_hint));
searchView.setOnQueryTextListener(new SearchView.OnQueryTextListener() {
@Override @Override
public boolean onMenuItemActionCollapse(MenuItem item) { public boolean onQueryTextChange(String newText) {
searching = false; searching = newText;
return true; return true;
} }
});
if (searching)
menuSearch.expandActionView();
final SearchView searchView = (SearchView) menuSearch.getActionView();
searchView.setQueryHint(getString(folder < 0 ? R.string.title_search_device : R.string.title_search_hint));
searchView.setOnQueryTextListener(new SearchView.OnQueryTextListener() {
@Override @Override
public boolean onQueryTextSubmit(String query) { public boolean onQueryTextSubmit(String query) {
searching = false; searching = null;
menuSearch.collapseActionView(); menuSearch.collapseActionView();
search(getContext(), getViewLifecycleOwner(), getFragmentManager(), folder, query); search(getContext(), getViewLifecycleOwner(), getFragmentManager(), folder, query);
return true; return true;
} }
@Override
public boolean onQueryTextChange(String newText) {
return false;
}
}); });
super.onCreateOptionsMenu(menu, inflater); super.onCreateOptionsMenu(menu, inflater);

@ -0,0 +1,11 @@
<?xml version="1.0" encoding="utf-8"?>
<menu xmlns:android="http://schemas.android.com/apk/res/android"
xmlns:app="http://schemas.android.com/apk/res-auto">
<item
android:id="@+id/menu_search"
android:icon="@drawable/baseline_search_24"
android:title="@string/title_search"
app:actionViewClass="androidx.appcompat.widget.SearchView"
app:showAsAction="collapseActionView|always" />
</menu>
Loading…
Cancel
Save