More information on Applications

Summary: More Information on Applications on Applications List View. Also, added tags in Applications Details view to show their status.

Test Plan: Manual Checking

Reviewers: epriestley

CC: aran, Korvin

Differential Revision: https://secure.phabricator.com/D4759

Author: Afaque Hussain

src/applications/meta/controller/PhabricatorApplicationDetailViewController.php

@@ -27,36 +27,53 @@ public function processRequest() {
         ->setName(pht('Applications'))
         ->setHref($this->getApplicationURI()));
 
-   $properties = $this->buildPropertyView($selected);
-   $actions = $this->buildActionView($user, $selected);
-
-   return $this->buildApplicationPage(
-    array(
-      $crumbs,
-      id(new PhabricatorHeaderView())->setHeader($title),
-      $actions,
-      $properties,
-    ),
-    array(
-      'title' => $title,
-      'device' => true,
-    ));
-  }
+    $header = id(new PhabricatorHeaderView())
+      ->setHeader($title);
 
-  private function buildPropertyView(PhabricatorApplication $selected) {
-    $properties = new PhabricatorPropertyListView();
+    $status_tag = id(new PhabricatorTagView())
+            ->setType(PhabricatorTagView::TYPE_STATE);
 
     if ($selected->isInstalled()) {
-      $properties->addProperty(
-        pht('Status'), pht('Installed'));
+      $status_tag->setName(pht('Installed'));
+      $status_tag->setBackgroundColor(PhabricatorTagView::COLOR_GREEN);
 
     } else {
-      $properties->addProperty(
-        pht('Status'), pht('Uninstalled'));
+      $status_tag->setName(pht('Uninstalled'));
+      $status_tag->setBackgroundColor(PhabricatorTagView::COLOR_RED);
     }
 
-    $properties->addProperty(
-      pht('Description'), $selected->getShortDescription());
+    if ($selected->isBeta()) {
+      $beta_tag = id(new PhabricatorTagView())
+              ->setType(PhabricatorTagView::TYPE_STATE)
+              ->setName(pht('Beta'))
+              ->setBackgroundColor(PhabricatorTagView::COLOR_GREY);
+      $header->addTag($beta_tag);
+    }
+
+
+    $header->addTag($status_tag);
+
+    $properties = $this->buildPropertyView($selected);
+    $actions = $this->buildActionView($user, $selected);
+
+    return $this->buildApplicationPage(
+      array(
+        $crumbs,
+        $header,
+        $actions,
+        $properties,
+      ),
+      array(
+        'title' => $title,
+        'device' => true,
+      ));
+  }
+
+  private function buildPropertyView(PhabricatorApplication $selected) {
+    $properties = id(new PhabricatorPropertyListView())
+              ->addProperty(
+                pht('Description'), $selected->getShortDescription()
+                );
 
     return $properties;
   }

src/applications/meta/controller/PhabricatorApplicationsListController.php

@@ -53,7 +53,16 @@ private function buildInstalledApplicationsList(array $applications) {
           ->setHref('/applications/view/'.get_class($application).'/')
           ->addAttribute(
             phutil_escape_html($application->getShortDescription()));
+
+        if (!$application->isInstalled()) {
+          $item->addIcon('delete', pht('Uninstalled'));
+        }
+
+        if ($application->isBeta()) {
+          $item->addIcon('lint-warning', pht('Beta'));
+        }
         $list->addItem($item);
+
       }
     return $list;
    }