AP-433 solves an animation bug that uncorreclty show the search view (#…

…161)
cliqz-oss · Jul 15, 2019 · 6188992 · 6188992
1 parent 57dbc2b
commit 6188992
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 2 deletions.
diff --git a/app/src/lumen/res/layout/search_bar_widget.xml b/app/src/lumen/res/layout/search_bar_widget.xml
@@ -19,6 +19,7 @@
 
     <TextView
         android:id="@+id/title_bar"
+        android:focusable="true"
         android:layout_width="0dp"
         android:layout_weight="1"
         android:layout_height="@dimen/title_bar_h"

diff --git a/app/src/main/java/com/cliqz/browser/widget/SearchBar.java b/app/src/main/java/com/cliqz/browser/widget/SearchBar.java
@@ -226,7 +226,9 @@ public void showTitleBar() {
         animation.setAnimationListener(new Animation.AnimationListener() {
             @Override
             public void onAnimationStart(Animation animation) {
-                searchEditText.clearFocus();
+                // We can't searchEditText.clearFocus() because it first requests focus bringing
+                // the SearchView to front.
+                titleBar.requestFocus();
             }
 
             @Override

diff --git a/build.gradle b/build.gradle
@@ -8,7 +8,7 @@ buildscript {
         }
     }
     dependencies {
-        classpath 'com.android.tools.build:gradle:3.4.1'
+        classpath 'com.android.tools.build:gradle:3.4.2'
         classpath 'com.getkeepsafe.dexcount:dexcount-gradle-plugin:0.8.6'
         classpath 'com.google.gms:google-services:4.2.0'
         classpath 'com.github.cliqz-oss:minibloomfilter:1.0'