From cffa4e41cf0cfdb929770c387720bb23c9dd15ed Mon Sep 17 00:00:00 2001
From: M66B <marcel@faircode.eu>
Date: Sun, 31 Aug 2014 18:00:15 +0200
Subject: [PATCH] Showing appropriate main and details help

Fixes #1921
---
 CHANGELOG.md                                |  3 ++-
 res/layout/legend.xml                       | 17 +++++++++++++----
 src/biz/bokhorst/xprivacy/ActivityApp.java  |  6 ++++++
 src/biz/bokhorst/xprivacy/ActivityMain.java |  6 ++++++
 4 files changed, 27 insertions(+), 5 deletions(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index d7bc8d665..b20382f1f 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -45,7 +45,8 @@ Version 3.x will be available with a [pro license](http://www.xprivacy.eu/) only
 
 **Next release**
 
-* Show application name above usage data ([issue](/../../issues/1949))
+* Showing appropriate main and details help ([issue](/../../issues/1921))
+* Showing application name above usage data ([issue](/../../issues/1949))
 
 [Open issues](https://github.com/M66B/XPrivacy/issues?state=open)
 
diff --git a/res/layout/legend.xml b/res/layout/legend.xml
index dbcf8cd9e..89bf141b9 100644
--- a/res/layout/legend.xml
+++ b/res/layout/legend.xml
@@ -34,6 +34,7 @@
             android:layout_width="wrap_content"
             android:layout_height="wrap_content"
             android:layout_marginTop="6dip"
+            android:tag="details"
             android:text="@string/help_switch"
             android:textAppearance="?android:attr/textAppearanceMedium"
             android:textIsSelectable="false" />
@@ -42,6 +43,7 @@
             android:layout_width="wrap_content"
             android:layout_height="wrap_content"
             android:layout_marginTop="6dip"
+            android:tag="main"
             android:text="@string/help_longpress"
             android:textAppearance="?android:attr/textAppearanceMedium"
             android:textIsSelectable="false" />
@@ -165,6 +167,7 @@
             android:drawableLeft="?attr/icon_info_book"
             android:drawablePadding="6dip"
             android:gravity="center_vertical"
+            android:tag="details"
             android:text="@string/title_function_help"
             android:textAppearance="?android:attr/textAppearanceMedium"
             android:textIsSelectable="false" />
@@ -175,6 +178,7 @@
             android:drawableLeft="?attr/icon_whitelist"
             android:drawablePadding="6dip"
             android:gravity="center_vertical"
+            android:tag="details"
             android:text="@string/menu_whitelists"
             android:textAppearance="?android:attr/textAppearanceMedium"
             android:textIsSelectable="false" />
@@ -186,6 +190,7 @@
             android:layout_width="wrap_content"
             android:layout_height="wrap_content"
             android:orientation="horizontal"
+            android:tag="details"
             android:visibility="gone" >
 
             <LinearLayout
@@ -216,7 +221,8 @@
         <LinearLayout
             android:layout_width="wrap_content"
             android:layout_height="wrap_content"
-            android:orientation="horizontal" >
+            android:orientation="horizontal"
+            android:tag="main" >
 
             <LinearLayout
                 android:layout_width="48dip"
@@ -270,7 +276,8 @@
             android:layout_width="wrap_content"
             android:layout_height="wrap_content"
             android:orientation="horizontal"
-            android:paddingTop="2dip" >
+            android:paddingTop="2dip"
+            android:tag="main" >
 
             <View
                 android:layout_width="12dip"
@@ -293,7 +300,8 @@
             android:layout_width="wrap_content"
             android:layout_height="wrap_content"
             android:orientation="horizontal"
-            android:paddingTop="2dip" >
+            android:paddingTop="2dip"
+            android:tag="main" >
 
             <View
                 android:layout_width="12dip"
@@ -316,7 +324,8 @@
             android:layout_width="wrap_content"
             android:layout_height="wrap_content"
             android:orientation="horizontal"
-            android:paddingTop="2dip" >
+            android:paddingTop="2dip"
+            android:tag="main" >
 
             <View
                 android:layout_width="12dip"
diff --git a/src/biz/bokhorst/xprivacy/ActivityApp.java b/src/biz/bokhorst/xprivacy/ActivityApp.java
index b58fd2295..8def3ff46 100644
--- a/src/biz/bokhorst/xprivacy/ActivityApp.java
+++ b/src/biz/bokhorst/xprivacy/ActivityApp.java
@@ -696,10 +696,16 @@ private void optionLegend() {
 		dialog.setTitle(R.string.menu_legend);
 		dialog.setContentView(R.layout.legend);
 		dialog.setFeatureDrawableResource(Window.FEATURE_LEFT_ICON, getThemed(R.attr.icon_launcher));
+
 		((ImageView) dialog.findViewById(R.id.imgHelpHalf)).setImageBitmap(getHalfCheckBox());
 		((ImageView) dialog.findViewById(R.id.imgHelpOnDemand)).setImageBitmap(getOnDemandCheckBox());
+
+		for (View child : Util.getViewsByTag((ViewGroup) dialog.findViewById(android.R.id.content), "main"))
+			child.setVisibility(View.GONE);
+
 		((LinearLayout) dialog.findViewById(R.id.llUnsafe)).setVisibility(PrivacyManager.cVersion3 ? View.VISIBLE
 				: View.GONE);
+
 		dialog.setCancelable(true);
 		dialog.show();
 	}
diff --git a/src/biz/bokhorst/xprivacy/ActivityMain.java b/src/biz/bokhorst/xprivacy/ActivityMain.java
index e8b6e9d73..a66809611 100644
--- a/src/biz/bokhorst/xprivacy/ActivityMain.java
+++ b/src/biz/bokhorst/xprivacy/ActivityMain.java
@@ -1015,10 +1015,16 @@ private void optionLegend() {
 		dialog.setTitle(R.string.menu_legend);
 		dialog.setContentView(R.layout.legend);
 		dialog.setFeatureDrawableResource(Window.FEATURE_LEFT_ICON, getThemed(R.attr.icon_launcher));
+
 		((ImageView) dialog.findViewById(R.id.imgHelpHalf)).setImageBitmap(getHalfCheckBox());
 		((ImageView) dialog.findViewById(R.id.imgHelpOnDemand)).setImageBitmap(getOnDemandCheckBox());
+
+		for (View child : Util.getViewsByTag((ViewGroup) dialog.findViewById(android.R.id.content), "details"))
+			child.setVisibility(View.GONE);
+
 		((LinearLayout) dialog.findViewById(R.id.llUnsafe)).setVisibility(PrivacyManager.cVersion3 ? View.VISIBLE
 				: View.GONE);
+
 		dialog.setCancelable(true);
 		dialog.show();
 	}