improvements to app selection UI for better layout and error handling

app/src/main/java/org/torproject/android/ui/AppManager.java

@@ -53,11 +53,11 @@ public class AppManager extends AppCompatActivity implements OnCheckedChangeList
         setTitle(R.string.apps_mode);
         getSupportActionBar().setDisplayHomeAsUpEnabled(true);
         
-        Button buttonSelectAll, buttonSelectNone, buttonInvert;
+        View buttonSelectAll, buttonSelectNone, buttonInvert;
 
-        buttonSelectAll =   (Button) findViewById(R.id.button_proxy_all);
-        buttonSelectNone =  (Button) findViewById(R.id.button_proxy_none);
-        buttonInvert =      (Button) findViewById(R.id.button_invert_selection);
+        buttonSelectAll =   findViewById(R.id.button_proxy_all);
+        buttonSelectNone =  findViewById(R.id.button_proxy_none);
+        buttonInvert =      findViewById(R.id.button_invert_selection);
 
         buttonSelectAll.setOnClickListener(new OnAutoClickListener(0));
         buttonSelectNone.setOnClickListener(new OnAutoClickListener(1));

app/src/main/res/layout/layout_apps.xml

@@ -3,7 +3,6 @@
         android:layout_width="match_parent"
         android:layout_height="match_parent"
         xmlns:android="http://schemas.android.com/apk/res/android"
-        android:orientation="vertical"
         android:duplicateParentState="true">
 
     <!-- 
@@ -25,14 +24,14 @@
 
     </ListView>
 
-    <RelativeLayout
+    <LinearLayout
         android:id="@+id/layout_button_filters"
         android:layout_width="match_parent"
         android:layout_height="wrap_content"
         android:layout_alignParentBottom="true"
         android:orientation="horizontal" >
 
-        <Button
+        <android.support.v7.widget.AppCompatImageButton
             android:id="@+id/button_proxy_all"
             android:layout_width="wrap_content"
             android:layout_height="wrap_content"
@@ -40,26 +39,31 @@
             android:layout_alignParentStart="true"
             android:layout_toLeftOf="@+id/button_proxy_none"
             android:layout_toStartOf="@+id/button_proxy_none"
-            android:text="@string/button_proxy_all" />
 
-        <Button
+            android:layout_weight="1"
+            android:src="@drawable/ic_playlist_add_check_black_36dp"/>
+
+        <android.support.v7.widget.AppCompatImageButton
             android:id="@+id/button_invert_selection"
             android:layout_width="wrap_content"
             android:layout_height="wrap_content"
             android:layout_alignParentRight="true"
             android:layout_alignParentEnd="true"
             android:layout_alignParentTop="true"
-            android:text="@string/button_invert_selection" />
 
-        <Button
+            android:layout_weight="1"
+            android:src="@drawable/ic_autorenew_black_36dp"/>
+
+        <android.support.v7.widget.AppCompatImageButton
             android:id="@+id/button_proxy_none"
             android:layout_width="wrap_content"
             android:layout_height="wrap_content"
             android:layout_alignParentTop="true"
+            android:layout_weight="1"
             android:layout_toLeftOf="@+id/button_invert_selection"
             android:layout_toStartOf="@+id/button_invert_selection"
-            android:text="@string/button_proxy_none" />
+            android:src='@drawable/ic_filter_none_black_36dp' />
 
-    </RelativeLayout>
+    </LinearLayout>
 
 </RelativeLayout>

app/src/main/res/values/strings.xml

@@ -218,7 +218,7 @@
   <string name="polipo_version">Polipo v1.1.9: https://github.com/jech/polipo</string>
   <string name="iptables_version">IPtables v1.4.21: http://www.netfilter.org</string>
   <string name="openssl_version">OpenSSL v1.0.2j: http://www.openssl.org</string>
-  <string name="hidden_service_request">An app wants to open hidden server port %S to the Tor network. This is safe if you trust the app.</string>
+  <string name="hidden_service_request">An app wants to open hidden server port %1$s to the Tor network. This is safe if you trust the app.</string>
   <string name="found_existing_tor_process">found existing Tor process&#8230;</string>
   <string name="something_bad_happened">Something bad happened. Check the log</string>
   <string name="hidden_service_on">hidden service on:</string>