mirror of
https://we.phorge.it/source/phorge.git
synced 2024-11-20 05:42:40 +01:00
Fix Ponder missing menu items.
Summary: Not sure what happened here, maybe crumbs got clobbered. Test Plan: Tested 'Ask Question' on all pages. Tested mobile. Reviewers: epriestley, btrahan Reviewed By: epriestley CC: aran, Korvin Differential Revision: https://secure.phabricator.com/D5725
This commit is contained in:
parent
58e35518b6
commit
5bc684ad36
2 changed files with 11 additions and 8 deletions
|
@ -34,7 +34,7 @@ abstract class PonderController extends PhabricatorController {
|
|||
->addAction(
|
||||
id(new PhabricatorMenuItemView())
|
||||
->setName(pht('New Question'))
|
||||
->setHref('question/ask')
|
||||
->setHref('/ponder/question/ask')
|
||||
->setIcon('create'));
|
||||
|
||||
return $crumbs;
|
||||
|
|
|
@ -16,17 +16,22 @@ final class PonderFeedController extends PonderController {
|
|||
$this->answerOffset = $request->getInt('aoff');
|
||||
|
||||
$pages = array(
|
||||
'feed' => 'All Questions',
|
||||
'questions' => 'Your Questions',
|
||||
'answers' => 'Your Answers',
|
||||
'feed' => pht('All Questions'),
|
||||
'questions' => pht('Your Questions'),
|
||||
'answers' => pht('Your Answers'),
|
||||
);
|
||||
|
||||
$side_nav = $this->buildSideNavView();
|
||||
|
||||
$this->page = $side_nav->selectFilter($this->page, 'feed');
|
||||
|
||||
$title = $pages[$this->page];
|
||||
|
||||
$crumbs = $this->buildApplicationCrumbs($this->buildSideNavView());
|
||||
$crumbs->addCrumb(
|
||||
id(new PhabricatorCrumbView())
|
||||
->setName($title)
|
||||
->setHref($this->getApplicationURI()));
|
||||
$side_nav->setCrumbs($crumbs);
|
||||
|
||||
switch ($this->page) {
|
||||
case 'feed':
|
||||
case 'questions':
|
||||
|
@ -53,8 +58,6 @@ final class PonderFeedController extends PonderController {
|
|||
$view = $this->buildQuestionListView($questions);
|
||||
$view->setPager($pager);
|
||||
|
||||
$side_nav->appendChild(
|
||||
id(new PhabricatorHeaderView())->setHeader($title));
|
||||
$side_nav->appendChild($view);
|
||||
break;
|
||||
case 'answers':
|
||||
|
|
Loading…
Reference in a new issue