Page MenuHomePhabricator

Prevent hiding the PhabricatorProjectDetailsProfileMenuItem
ClosedPublic

Authored by 20after4 on Dec 22 2017, 3:24 PM.
Tags
None
Referenced Files
F19940377: D18836.diff
Wed, Apr 15, 1:02 AM
F19862502: D18836.id45204.diff
Mar 14 2026, 10:38 AM
F19055346: D18836.id.diff
Nov 28 2025, 5:04 PM
F19048293: D18836.diff
Nov 27 2025, 2:24 PM
F19037484: D18836.id45204.diff
Nov 25 2025, 11:42 PM
F19002421: D18836.id.diff
Nov 21 2025, 5:55 AM
F18835206: D18836.diff
Oct 26 2025, 2:13 PM
F18806894: D18836.diff
Oct 18 2025, 9:57 PM
Subscribers

Details

Summary

This probably isn't the best solution, however, it conveniently avoids the bug from T13033. It would probably be more user-friendly (but more difficult to implement) if we allowed either Project Details or Workboard to be hidden but not both.

Test Plan

Tested locally, indeed this prevents hiding the menu item.

Diff Detail

Repository
rP Phabricator
Branch
master
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 18969
Build 25583: Run Core Tests
Build 25582: arc lint + arc unit

Event Timeline

I think this is reasonable, hiding this doesn't make a ton of sense to me given that you can make something else the default anyway. I'll look at fixing T13033 properly, too.

This revision is now accepted and ready to land.Dec 22 2017, 3:43 PM

(I'm just going to pull this since it works alongside D18843.)

This revision was automatically updated to reflect the committed changes.

@epriestley: Thanks! Hope you're having a great holiday weekend :)