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
Unknown Object (File)
Fri, Jun 20, 2:24 PM
Unknown Object (File)
Thu, Jun 19, 3:32 PM
Unknown Object (File)
Apr 24 2025, 8:26 AM
Unknown Object (File)
Apr 9 2025, 6:07 PM
Unknown Object (File)
Mar 29 2025, 2:14 PM
Unknown Object (File)
Mar 27 2025, 5:19 PM
Unknown Object (File)
Mar 24 2025, 9:24 PM
Unknown Object (File)
Feb 24 2025, 3:06 AM

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.