Homec4science

Move Favorites and User menus to MenuBarExtensions

Authored by epriestley <git@epriestley.com> on Jan 21 2017, 17:11.

Description

Move Favorites and User menus to MenuBarExtensions

Summary:
Ref T12140. The major effect of this change is that uninstalling "Home" (as we do on admin.phacility.com) no longer uninstalls the user menu (which is required to access settings or log out).

This also simplifies the code a bit, by consolidating how menus are built into MenuBarExtensions instead of some in Applications and some in Extensions.

Test Plan:

  • While logged in and logged out, saw main menus in the correct order.
  • Uninstalled Favorites, saw the menu vanish.
  • Uninstalled Home, still had a user menu.

Reviewers: chad

Reviewed By: chad

Maniphest Tasks: T12140

Differential Revision: https://secure.phabricator.com/D17239

Details

Committed
epriestley <git@epriestley.com>Jan 21 2017, 17:50
Pushed
aubortJan 31 2017, 17:16
Parents
rPHddf82a815b9a: Remove duplicate setIsRequired()
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPH402b6473d809: Move Favorites and User menus to MenuBarExtensions (authored by epriestley <git@epriestley.com>).Jan 21 2017, 17:50