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)
Sat, Feb 15, 6:52 AM
Unknown Object (File)
Fri, Feb 14, 3:56 AM
Unknown Object (File)
Thu, Feb 6, 9:09 AM
Unknown Object (File)
Thu, Feb 6, 8:30 AM
Unknown Object (File)
Mon, Feb 3, 9:09 AM
Unknown Object (File)
Fri, Jan 31, 4:56 PM
Unknown Object (File)
Jan 22 2025, 3:13 PM
Unknown Object (File)
Dec 28 2024, 4:48 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.