1
0
Fork 0
mirror of https://we.phorge.it/source/phorge.git synced 2024-11-22 14:52:41 +01:00

Fix double-menus in Herald transcripts

Summary: Fixes T2329. Adds crumbs and gets rid of the double-depth menus.

Test Plan: Looked at transcript list, test controller and transcript detail.

Reviewers: chad, btrahan

Reviewed By: chad

CC: aran

Maniphest Tasks: T2329

Differential Revision: https://secure.phabricator.com/D4564
This commit is contained in:
epriestley 2013-01-21 07:46:13 -08:00
parent be91064a1d
commit 73f63158a8
3 changed files with 26 additions and 4 deletions

View file

@ -125,6 +125,16 @@ final class HeraldTestConsoleController extends HeraldController {
$panel,
));
$crumbs = id($this->buildApplicationCrumbs())
->addCrumb(
id(new PhabricatorCrumbView())
->setName(pht('Transcripts'))
->setHref($this->getApplicationURI('/transcript/')))
->addCrumb(
id(new PhabricatorCrumbView())
->setName(pht('Test Console')));
$nav->setCrumbs($crumbs);
return $this->buildStandardPageResponse(
$nav,
array(

View file

@ -70,12 +70,18 @@ final class HeraldTranscriptController extends HeraldController {
$nav->appendChild($object_xscript_panel);
}
$main_nav = $this->renderNav();
$main_nav->selectFilter('transcript');
$main_nav->appendChild($nav);
$crumbs = id($this->buildApplicationCrumbs())
->addCrumb(
id(new PhabricatorCrumbView())
->setName(pht('Transcripts'))
->setHref($this->getApplicationURI('/transcript/')))
->addCrumb(
id(new PhabricatorCrumbView())
->setName($xscript->getID()));
$nav->setCrumbs($crumbs);
return $this->buildStandardPageResponse(
$main_nav,
$nav,
array(
'title' => 'Transcript',
));

View file

@ -99,6 +99,12 @@ final class HeraldTranscriptListController extends HeraldController {
$nav->selectFilter('transcript');
$nav->appendChild($panel);
$crumbs = id($this->buildApplicationCrumbs())
->addCrumb(
id(new PhabricatorCrumbView())
->setName(pht('Transcripts')));
$nav->setCrumbs($crumbs);
return $this->buildStandardPageResponse(
$nav,
array(