Slightly better layout of app packages + pseudo icons for permission protection level.

diff --git a/wp-fdroid/wp-fdroid.php b/wp-fdroid/wp-fdroid.php
index 34bc730a1f13c830906a711502e1e113030aff05..7b33e4ff5d8516499737199abe75fc02fd86bdbc 100644 (file)
--- a/wp-fdroid/wp-fdroid.php
+++ b/wp-fdroid/wp-fdroid.php
@@ -209,18 +209,18 @@ class FDroid
                 $out.="<h3>Packages</h3>";\r
                                $i=0;\r
                 foreach($apks as $apk) {\r
-                    $out.="<p><b>Version ".$apk['version']."</b> - ";\r
-                    $out.='<a href="http://f-droid.org/repo/'.$apk['apkname'].'">download</a> ';\r
+                    $out.="<p><b>Version ".$apk['version']."</b><br />";\r
+                    $out.='<a href="http://f-droid.org/repo/'.$apk['apkname'].'">download apk</a> ';\r
                     $out.=$apk['size']." bytes";\r
                     if($apk['srcname'])\r
                         $out.='<br><a href="http://f-droid.org/repo/'.$apk['srcname'].'">source tarball</a>';\r
                                        \r
-                                       if($i==0)\r
+                                       /*if($i==0)\r
                                                $divStyleDisplay='block';\r
-                                       else\r
+                                       else*/\r
                                                $divStyleDisplay='none';\r
                                        $divId='permissions'.$i;\r
-                                       $out.='<p><a href="javascript:void(0);" onClick="showHidePermissions(\''.$divId.'\');">Permissions</a><br/>';\r
+                                       $out.='<br /><a href="javascript:void(0);" onClick="showHidePermissions(\''.$divId.'\');">view permissions</a><br/>';\r
                                        $out.='<div style="display:'.$divStyleDisplay.';" id="'.$divId.'">';\r
                                        $permissions = explode(',',$apk['permissions']);\r
                                        usort($permissions,\r
@@ -253,13 +253,13 @@ class FDroid
                                                        $permission_group_last = $permission_group;\r
                                                }\r
                                        \r
-                                               $out.='('.strtoupper(substr($permissions_data['permission'][$permission]['protectionLevel'],0,1)).') - ';\r
+                                               $out.=$this->get_permission_protection_level_icon($permissions_data['permission'][$permission]['protectionLevel']).' ';\r
                                                $out.='<strong>'.$permissions_data['permission'][$permission]['label'].'</strong> [<code>'.$permission.'</code>]<br/>';\r
                                                $out.=$permissions_data['permission'][$permission]['description'].'<br/>';\r
                                                //$out.=$permissions_data['permission'][$permission]['comment'].'<br/>';\r
                                                $out.='<br/>';\r
                                        }\r
-                                       $out.='</div></p>';\r
+                                       $out.='</div>';\r
                                                \r
                     $out.='</p>';\r
                                        $i++;\r
@@ -273,6 +273,20 @@ class FDroid
         return "<p>Application not found</p>";\r
     }\r
 \r
+       private function get_permission_protection_level_icon($protection_level) {\r
+               if($protection_level=='dangerous')\r
+               {\r
+                       return '<span style="color:#DD9900;font-size:150%;">&#x26a0;</span>';\r
+               }\r
+               elseif($protection_level=='normal')\r
+               {\r
+                       return '<span style="color:#6666FF;font-size:110%;">&#x24D8;</span>';\r
+               }\r
+               else\r
+               {\r
+                       return '<em>!</em>';\r
+               }\r
+       }       \r
 \r
     function get_apps($query_vars) {\r
 \r