Page MenuHomePhabricator

Fix missing button to expand project action list on mobile.
ClosedPublic

Authored by brechtvl on Nov 27 2013, 7:58 PM.
Tags
None
Referenced Files
F18812689: D7666.id17311.diff
Mon, Oct 20, 11:32 AM
F18779931: D7666.diff
Sat, Oct 11, 3:52 PM
F18099965: D7666.id17311.diff
Aug 9 2025, 9:47 AM
F18097349: D7666.id.diff
Aug 8 2025, 12:59 PM
F18093297: D7666.diff
Aug 7 2025, 5:15 PM
F17944324: D7666.id17310.diff
Jul 31 2025, 9:54 AM
F17939831: D7666.diff
Jul 31 2025, 3:29 AM
F17935860: D7666.id.diff
Jul 30 2025, 11:40 PM

Details

Test Plan

Go to a project page, make browser window narrow, click to expand action list.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

This is the slightly more standard/general fix, does that work?

diff --git a/src/applications/project/controller/PhabricatorProjectController.php b/src/applications/project/controller/PhabricatorProjectController.php
index a50d0af..8a72300 100644
--- a/src/applications/project/controller/PhabricatorProjectController.php
+++ b/src/applications/project/controller/PhabricatorProjectController.php
@@ -21,6 +21,10 @@ abstract class PhabricatorProjectController extends PhabricatorController {
     return $nav;
   }
 
+  public function buildApplicationMenu() {
+    return $this->buildSideNavView(true)->getMenu();
+  }
+
   public function buildApplicationCrumbs() {
     $crumbs = parent::buildApplicationCrumbs();

Oh -- I think the two do different things. Let me check...

Yeah, one fixes actions ("Edit Project...") one fixes the availability of the navigation menu.

I think we're going to pull the action element back out of the header, but this is fine for the moment.