From 759ecbc7b82be80261164d392875bb237764ed6f Mon Sep 17 00:00:00 2001
From: agrajaghh <agrajaghh@gmail.com>
Date: Fri, 25 Dec 2015 13:51:51 +0100
Subject: [PATCH] completely hide location button on API < 16

Closes #4964
// FREEBIE
---
 res/layout/attachment_type_selector.xml       | 37 ++++++++-----------
 .../components/AttachmentTypeSelector.java    |  2 +-
 2 files changed, 16 insertions(+), 23 deletions(-)

diff --git a/res/layout/attachment_type_selector.xml b/res/layout/attachment_type_selector.xml
index 85a8b294e8..9b912d0122 100644
--- a/res/layout/attachment_type_selector.xml
+++ b/res/layout/attachment_type_selector.xml
@@ -138,34 +138,27 @@
 
         </LinearLayout>
 
-        <LinearLayout android:layout_width="match_parent"
+        <LinearLayout android:id="@+id/location_linear_layout"
+                      android:layout_width="match_parent"
                       android:layout_height="wrap_content"
                       android:layout_weight="1"
                       android:gravity="center"
                       android:orientation="vertical">
 
-            <LinearLayout android:layout_width="match_parent"
-                          android:layout_height="wrap_content"
-                          android:layout_weight="1"
-                          android:gravity="center"
-                          android:orientation="vertical">
+            <org.thoughtcrime.securesms.components.CircleColorImageView
+                    android:id="@+id/location_button"
+                    android:layout_width="60dp"
+                    android:layout_height="60dp"
+                    android:src="@drawable/ic_location_on_white_36dp"
+                    android:scaleType="center"
+                    android:elevation="4dp"
+                    app:circleColor="@color/blue_grey_400"/>
 
-                <org.thoughtcrime.securesms.components.CircleColorImageView
-                        android:id="@+id/location_button"
-                        android:layout_width="60dp"
-                        android:layout_height="60dp"
-                        android:src="@drawable/ic_location_on_white_36dp"
-                        android:scaleType="center"
-                        android:elevation="4dp"
-                        app:circleColor="@color/blue_grey_400"/>
-
-                <TextView android:layout_marginTop="10dp"
-                          android:layout_width="wrap_content"
-                          android:layout_height="wrap_content"
-                          style="@style/AttachmentTypeLabel"
-                          android:text="@string/attachment_type_selector__location"/>
-
-            </LinearLayout>
+            <TextView android:layout_marginTop="10dp"
+                      android:layout_width="wrap_content"
+                      android:layout_height="wrap_content"
+                      style="@style/AttachmentTypeLabel"
+                      android:text="@string/attachment_type_selector__location"/>
 
         </LinearLayout>
 
diff --git a/src/org/thoughtcrime/securesms/components/AttachmentTypeSelector.java b/src/org/thoughtcrime/securesms/components/AttachmentTypeSelector.java
index e37605d7a5..f86b3b496f 100644
--- a/src/org/thoughtcrime/securesms/components/AttachmentTypeSelector.java
+++ b/src/org/thoughtcrime/securesms/components/AttachmentTypeSelector.java
@@ -74,7 +74,7 @@ public class AttachmentTypeSelector extends PopupWindow {
     this.closeButton.setOnClickListener(new CloseClickListener());
 
     if (Build.VERSION.SDK_INT < Build.VERSION_CODES.JELLY_BEAN) {
-      this.locationButton.setVisibility(View.INVISIBLE);
+      ViewUtil.findById(layout, R.id.location_linear_layout).setVisibility(View.INVISIBLE);
     }
 
     setContentView(layout);