Page MenuHomePhabricator

Prevent hiding the PhabricatorProjectDetailsProfileMenuItem
ClosedPublic

Authored by 20after4 on Dec 22 2017, 3:24 PM.
Tags
None
Referenced Files
F13240824: D18836.id45192.diff
Wed, May 22, 4:53 PM
F13236382: D18836.diff
Tue, May 21, 9:29 AM
F13233240: D18836.diff
Tue, May 21, 2:08 AM
F13228881: D18836.id45192.diff
Mon, May 20, 12:53 PM
F13210254: D18836.id.diff
Fri, May 17, 4:39 AM
F13189373: D18836.diff
Sat, May 11, 5:54 AM
Unknown Object (File)
Mon, May 6, 7:12 PM
Unknown Object (File)
Thu, May 2, 9:12 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
Lint
Lint Not Applicable
Unit
Tests Not Applicable

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 :)