From 3b735de1525313b0517473aff849914b1722a445 Mon Sep 17 00:00:00 2001 From: calderwoodra Date: Thu, 18 Jan 2018 14:03:17 -0800 Subject: Fixed regression with search not reacting properly to on touch events. There are a few interesting UX niceties that we support in search: 1) When in regular search with an empty query, close the UI if the user touches the blank space. 2) When in regular search with a non-empty query, hide the keyboard if the user touches the the list so they can see all results. 2) When in dialpad search with an empty query, close the UI if the user touches the blank space. 3) When in dialpad search with a non-empty query, hide the dialpad so the user can see the full list of results. This change also adds logic to transfer the dialpad query to the search bar. Bug: 64655802 Test: MainActivityIntegrationTest PiperOrigin-RevId: 182434126 Change-Id: Iabb73b0018fa20e2811010a73a35d3aa3b35343b --- java/com/android/dialer/searchfragment/list/NewSearchFragment.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'java/com/android/dialer/searchfragment') diff --git a/java/com/android/dialer/searchfragment/list/NewSearchFragment.java b/java/com/android/dialer/searchfragment/list/NewSearchFragment.java index e8a8a4e06..30949d38f 100644 --- a/java/com/android/dialer/searchfragment/list/NewSearchFragment.java +++ b/java/com/android/dialer/searchfragment/list/NewSearchFragment.java @@ -490,8 +490,8 @@ public final class NewSearchFragment extends Fragment public boolean onTouch(View v, MotionEvent event) { if (event.getAction() == MotionEvent.ACTION_UP) { v.performClick(); - FragmentUtils.getParentUnsafe(this, SearchFragmentListener.class).onSearchListTouch(); } + FragmentUtils.getParentUnsafe(this, SearchFragmentListener.class).onSearchListTouch(); return false; } -- cgit v1.2.3