From cdcd49e4eece0d16d0ca0588614e4114820eec53 Mon Sep 17 00:00:00 2001
From: Anh Nhan Nguyen <anhnhan@outlook.com>
Date: Mon, 01 Apr 2013 13:28:38 +0000
Subject: [PATCH] Revising Flag UI

Summary:
Re-adding side nav (feels less spacy now).

Replaced header with crumb. Looks nicer with the filter imo.

Test Plan: {F38201}

Reviewers: epriestley

Reviewed By: epriestley

CC: aran, Korvin

Differential Revision: https://secure.phabricator.com/D5517
---
 src/applications/flag/controller/PhabricatorFlagListController.php | 16 +++++++++++-----
 1 file(s) changed, 11 insertion(s)(+), 5 deletion(s)(-)

diff --git a/src/applications/flag/controller/PhabricatorFlagListController.php b/src/applications/flag/controller/PhabricatorFlagListController.php
--- a/src/applications/flag/controller/PhabricatorFlagListController.php
+++ b/src/applications/flag/controller/PhabricatorFlagListController.php
@@ -7,6 +7,16 @@
     $user = $request->getUser();
 
     $nav = new AphrontSideNavFilterView();
+    $nav->setBaseURI(new PhutilURI('/flag/view/'));
+    $nav->addLabel(pht('Flags'));
+    $nav->addFilter('all', pht('Your Flags'));
+    $nav->selectFilter('all', 'all');
+
+    $crumbs = $this->buildApplicationCrumbs();
+    $crumbs->addCrumb(id(new PhabricatorCrumbView)
+      ->setName(pht('Flags'))
+      ->setHref($request->getRequestURI()));
+    $nav->setCrumbs($crumbs);
 
     $filter_form = new AphrontFormView();
     $filter_form->setUser($user);
@@ -26,6 +36,7 @@
 
     $filter = new AphrontListFilterView();
     $filter->appendChild($filter_form);
+    $nav->appendChild($filter);
 
     $query = new PhabricatorFlagQuery();
     $query->withOwnerPHIDs(array($user->getPHID()));
@@ -56,11 +67,6 @@
     $view->setFlags($flags);
     $view->setUser($user);
 
-    $header = new PhabricatorHeaderView();
-    $header->setHeader(pht('Flags'));
-
-    $nav->appendChild($header);
-    $nav->appendChild($filter);
     $nav->appendChild($view);
 
     return $this->buildApplicationPage(

--
tangled.sh