Improved function help layout

Refs #1947

CHANGELOG.md

@@ -46,7 +46,7 @@ Version 3.x will be available with a [pro license](http://www.xprivacy.eu/) only
 **Next release**
 
 * Improved settings layout ([pull request](/../../issues/1946))
-* Show function help in a dialog ([pull request](/../../issues/1947))
+* Improved function help layout ([pull request](/../../issues/1947))
 
 [Open issues](https://github.com/M66B/XPrivacy/issues?state=open)
 

res/layout/help.xml

@@ -1,5 +1,6 @@
 <?xml version="1.0" encoding="utf-8"?>
 <ScrollView xmlns:android="http://schemas.android.com/apk/res/android"
+    xmlns:tools="http://schemas.android.com/tools"
     android:layout_width="match_parent"
     android:layout_height="wrap_content"
     android:orientation="vertical"
@@ -17,13 +18,30 @@
             android:id="@+id/tvTitle"
             android:layout_width="wrap_content"
             android:layout_height="wrap_content"
+            android:textAppearance="?android:attr/textAppearanceMedium"
             android:textStyle="bold" />
 
         <TextView
             android:id="@+id/tvInfo"
             android:layout_width="wrap_content"
             android:layout_height="wrap_content"
-            android:layout_marginTop="12dip" />
+            android:layout_marginTop="6dip" />
+
+        <TextView
+            android:layout_width="wrap_content"
+            android:layout_height="wrap_content"
+            android:layout_marginTop="12dip"
+            android:text="@string/title_permissions"
+            android:textAppearance="?android:attr/textAppearanceMedium"
+            android:textStyle="bold" />
+
+        <TextView
+            android:id="@+id/tvPermissions"
+            android:layout_width="wrap_content"
+            android:layout_height="wrap_content"
+            android:layout_marginTop="6dip"
+            android:text="-"
+            tools:ignore="HardcodedText" />
     </LinearLayout>
 
 </ScrollView>

src/biz/bokhorst/xprivacy/ActivityApp.java

@@ -1783,25 +1783,23 @@ public static void showHelp(ActivityBase context, View parent, Hook hook) {
 		dlgHelp.setFeatureDrawableResource(Window.FEATURE_LEFT_ICON, context.getThemed(R.attr.icon_launcher));
 		dlgHelp.setCancelable(true);
 
-		// Set text title
+		// Set title
 		TextView tvTitle = (TextView) dlgHelp.findViewById(R.id.tvTitle);
 		tvTitle.setText(hook.getName());
 
-		// Set text content
-		String text = hook.getAnnotation();
-		String[] permissions = hook.getPermissions();
-		if (permissions != null && permissions.length > 0) {
-			text += "<br /><br /><b>" + context.getString(R.string.title_permissions) + "</b><br /><br />";
-			if (permissions[0].equals(""))
-				text += "-";
-			else
-				text += TextUtils.join("<br />", permissions);
-		}
-
+		// Set info
 		TextView tvInfo = (TextView) dlgHelp.findViewById(R.id.tvInfo);
-		tvInfo.setText(Html.fromHtml(text));
+		tvInfo.setText(Html.fromHtml(hook.getAnnotation()));
 		tvInfo.setMovementMethod(LinkMovementMethod.getInstance());
 
+		// Set permissions
+		String[] permissions = hook.getPermissions();
+		if (permissions != null && permissions.length > 0)
+			if (!permissions[0].equals("")) {
+				TextView tvPermissions = (TextView) dlgHelp.findViewById(R.id.tvPermissions);
+				tvPermissions.setText(Html.fromHtml(TextUtils.join("<br />", permissions)));
+			}
+
 		dlgHelp.show();
 	}
 }