Page MenuHomePhabricator

Begin modularizing main menu items
ClosedPublic

Authored by epriestley on Jan 1 2016, 1:16 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Mar 5, 7:11 PM
Unknown Object (File)
Tue, Mar 5, 5:19 PM
Unknown Object (File)
Feb 3 2024, 8:10 AM
Unknown Object (File)
Jan 28 2024, 12:11 PM
Unknown Object (File)
Jan 23 2024, 3:46 PM
Unknown Object (File)
Jan 19 2024, 1:07 PM
Unknown Object (File)
Dec 27 2023, 8:01 AM
Unknown Object (File)
Dec 27 2023, 8:01 AM
Subscribers
None

Details

Summary

Ref T10077. Ref T8918. The way the main menu is built is not very modular and fairly hacky.

It assumes menus are provided by applications, but this isn't exactly true. Notably, the "Quick Create" menu is not per-application.

The current method of building this menu is very inefficient (see T10077). Particularly, we have to build it twice because we need to build it once to render the item and then again to render the dropdown options.

Start cleaning this up. This diff doesn't actually have any behavioral changes, since I can't swap the menu over until we get rid of all the other items and I haven't extended this to Notifications/Conpherence yet so it doesn't actually fix T8918.

Test Plan

Viewed menus while logged in, logged out, in different applications, in desktop/mobile. Nothing appeared different.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Begin modularizing main menu items.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
chad edited edge metadata.
This revision is now accepted and ready to land.Jan 1 2016, 4:16 PM
This revision was automatically updated to reflect the committed changes.