Remove superfluous icon and padding that came with sdk 24

Fixes #5965 // FREEBIE
signalapp · Dec 17, 2016 · 4ff8de0 · 4ff8de0
1 parent d6d3d1f
commit 4ff8de0
Show file tree

Hide file tree

Showing 7 changed files with 14 additions and 35 deletions.
diff --git a/res/layout/contact_filter_toolbar.xml b/res/layout/contact_filter_toolbar.xml
@@ -5,15 +5,6 @@
                   android:layout_height="match_parent"
                   android:orientation="horizontal">
 
-        <ImageView android:id="@+id/action_icon"
-                   android:layout_width="wrap_content"
-                   android:layout_height="wrap_content"
-                   android:layout_gravity="center_vertical"
-                   android:gravity="center_vertical"
-                   android:clickable="true"
-                   android:background="@drawable/circle_touch_highlight_background"
-                   android:src="@drawable/ic_search_white_24dp" />
-
         <LinearLayout android:id="@+id/toggle_container"
                       android:layout_width="match_parent"
                       android:layout_height="match_parent"
@@ -23,8 +14,6 @@
                       android:layout_height="wrap_content"
                       android:layout_width="0px"
                       android:layout_weight="1"
-                      android:layout_marginLeft="5dp"
-                      android:layout_marginStart="5dp"
                       android:hint="@string/contact_selection_activity__enter_name_or_number"
                       android:inputType="textPersonName"
                       style="@style/TextSecure.TitleTextStyle"

diff --git a/res/layout/contact_selection_activity.xml b/res/layout/contact_selection_activity.xml
@@ -3,15 +3,17 @@
               android:layout_height="fill_parent"
               android:layout_width="fill_parent"
               android:orientation="vertical"
-              xmlns:android="http://schemas.android.com/apk/res/android">
+              xmlns:android="http://schemas.android.com/apk/res/android"
+              xmlns:app="http://schemas.android.com/apk/res-auto">
 
     <org.thoughtcrime.securesms.components.ContactFilterToolbar
             android:id="@+id/toolbar"
             android:layout_height="?attr/actionBarSize"
             android:layout_width="match_parent"
             android:minHeight="?attr/actionBarSize"
             android:background="?attr/colorPrimary"
-            android:theme="?attr/actionBarStyle" />
+            android:theme="?attr/actionBarStyle"
+            app:contentInsetStartWithNavigation="0dp"/>
 
         <fragment android:id="@+id/contact_selection_list_fragment"
                   android:layout_width="match_parent"

diff --git a/src/org/thoughtcrime/securesms/ContactSelectionActivity.java b/src/org/thoughtcrime/securesms/ContactSelectionActivity.java
@@ -96,6 +96,8 @@ private void initializeToolbar() {
 
     getSupportActionBar().setDisplayHomeAsUpEnabled(false);
     getSupportActionBar().setDisplayShowTitleEnabled(false);
+    getSupportActionBar().setIcon(null);
+    getSupportActionBar().setLogo(null);
   }
 
   private void initializeResources() {

diff --git a/src/org/thoughtcrime/securesms/InviteActivity.java b/src/org/thoughtcrime/securesms/InviteActivity.java
@@ -95,6 +95,7 @@ private void initializeResources() {
     smsCancelButton.setOnClickListener(new SmsCancelClickListener());
     smsSendButton.setOnClickListener(new SmsSendClickListener());
     contactFilter.setOnFilterChangedListener(new ContactFilterChangedListener());
+    contactFilter.setNavigationIcon(R.drawable.ic_search_white_24dp);
   }
 
   private Animation loadAnimation(@AnimRes int animResId) {

diff --git a/src/org/thoughtcrime/securesms/NewConversationActivity.java b/src/org/thoughtcrime/securesms/NewConversationActivity.java
@@ -44,7 +44,6 @@ public class NewConversationActivity extends ContactSelectionActivity {
   public void onCreate(Bundle bundle, @NonNull MasterSecret masterSecret) {
     super.onCreate(bundle, masterSecret);
 
-    getToolbar().setShowCustomNavigationButton(false);
     getSupportActionBar().setDisplayHomeAsUpEnabled(true);
   }
 

diff --git a/src/org/thoughtcrime/securesms/RecipientPreferenceActivity.java b/src/org/thoughtcrime/securesms/RecipientPreferenceActivity.java
@@ -130,6 +130,8 @@ public boolean onOptionsItemSelected(MenuItem item) {
 
   private void initializeToolbar() {
     this.toolbar = (Toolbar) findViewById(R.id.toolbar);
+    this.toolbar.setLogo(null);
+
     setSupportActionBar(toolbar);
 
     getSupportActionBar().setDisplayHomeAsUpEnabled(true);

diff --git a/src/org/thoughtcrime/securesms/components/ContactFilterToolbar.java b/src/org/thoughtcrime/securesms/components/ContactFilterToolbar.java
@@ -22,7 +22,6 @@ public class ContactFilterToolbar extends Toolbar {
 
   private EditText        searchText;
   private AnimatingToggle toggle;
-  private ImageView       action;
   private ImageView       keyboardToggle;
   private ImageView       dialpadToggle;
   private ImageView       clearToggle;
@@ -40,8 +39,6 @@ public ContactFilterToolbar(Context context, AttributeSet attrs, int defStyleAtt
     super(context, attrs, defStyleAttr);
     inflate(context, R.layout.contact_filter_toolbar, this);
 
-
-    this.action          = ViewUtil.findById(this, R.id.action_icon);
     this.searchText      = ViewUtil.findById(this, R.id.search_view);
     this.toggle          = ViewUtil.findById(this, R.id.button_toggle);
     this.keyboardToggle  = ViewUtil.findById(this, R.id.search_keyboard);
@@ -97,23 +94,10 @@ public void afterTextChanged(Editable s) {
       }
     });
 
-    expandTapArea(this, action);
-    expandTapArea(toggleContainer, dialpadToggle);
-  }
-
-  @Override
-  public void setNavigationIcon(int resId) {
-    action.setImageResource(resId);
-  }
-
-  @Override
-  public void setNavigationOnClickListener(OnClickListener listener) {
-    super.setNavigationOnClickListener(listener);
-    action.setOnClickListener(listener);
-  }
+    setLogo(null);
+    setContentInsetStartWithNavigation(0);
 
-  public void setShowCustomNavigationButton(boolean show) {
-    action.setVisibility(show ? VISIBLE : GONE);
+    expandTapArea(toggleContainer, dialpadToggle);
   }
 
   public void clear() {
@@ -154,11 +138,11 @@ public void run() {
   }
 
   private static class SearchUtil {
-    public static boolean isTextInput(EditText editText) {
+    static boolean isTextInput(EditText editText) {
       return (editText.getInputType() & InputType.TYPE_MASK_CLASS) == InputType.TYPE_CLASS_TEXT;
     }
 
-    public static boolean isPhoneInput(EditText editText) {
+    static boolean isPhoneInput(EditText editText) {
       return (editText.getInputType() & InputType.TYPE_MASK_CLASS) == InputType.TYPE_CLASS_PHONE;
     }