Page MenuHomePhabricator
Differential D11848

Make Project Mobile menu more resilient
ClosedPublic
Actions

Authored by chad on Feb 21 2015, 3:10 PM.
Tags
None
Referenced Files
F19153409: D11848.diff
Thu, Dec 11, 9:40 AM
F18945332: D11848.id28552.diff
Nov 11 2025, 6:06 PM
F18817302: D11848.id28580.diff
Oct 21 2025, 2:23 PM
F18673253: D11848.id28579.diff
Sep 25 2025, 9:53 AM
F18673252: D11848.id28552.diff
Sep 25 2025, 9:53 AM
F18445170: D11848.id.diff
Aug 31 2025, 8:14 PM
F18393924: D11848.diff
Aug 29 2025, 9:34 AM
F18110973: D11848.id28552.diff
Aug 12 2025, 3:46 AM
View All Files
Subscribers
epriestley
Korvin

Details

Reviewers
epriestley
btrahan
Maniphest Tasks
T7289: Projects sidebar does not appear on mobile
Commits
Restricted Diffusion Commit
rPb070ed50197f: Make Project Mobile menu more resilient
Summary

This sets $project at the controller level so it's available to building the icon nav and mobile menu, instead of guessing the $id. Fixes T7289

Test Plan

Test a project and its mobile menu when it does and does not have a workboard initialized. Also tested coming in via a slug and an id.

Diff Detail

Repository
rP Phabricator
Branch
projects-mobile-menu
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 4618
Build 4632: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

chad updated this revision to Diff 28552.Feb 21 2015, 3:10 PM
chad retitled this revision from to Make Project Mobile menu more resilient.
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added a reviewer: epriestley.
chad added a task: T7289: Projects sidebar does not appear on mobile.
Herald added subscribers: epriestley, Korvin. · View Herald TranscriptFeb 21 2015, 3:10 PM
chad added a reviewer: btrahan.Feb 23 2015, 5:27 PM
btrahan accepted this revision.Feb 23 2015, 5:33 PM
btrahan edited edge metadata.

Silly (anal?) style stuff in line.

src/applications/project/controller/PhabricatorProjectController.php
21

getProject()

29–30

just if ($project)

53

can just lose line 53

This revision is now accepted and ready to land.Feb 23 2015, 5:33 PM
chad updated this revision to Diff 28579.Feb 23 2015, 7:27 PM
chad edited edge metadata.
  • Update per comments, retest
Closed by commit rPb070ed50197f: Make Project Mobile menu more resilient (authored by chad, committed by chad). · Explain WhyFeb 23 2015, 7:27 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
applications/
project/
controller/
10 lines
21 lines

Diff 28579

View Options

src/applications/project/controller/PhabricatorProjectBoardController.php

Loading...
View Options

src/applications/project/controller/PhabricatorProjectController.php

Loading...
Phabricator · Built by Phacility