Page MenuHomePhabricator

Fully modularize the "Quick Actions" menu
ClosedPublic

Authored by epriestley on Jun 5 2016, 2:34 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Oct 9, 3:56 AM
Unknown Object (File)
Mon, Oct 7, 7:14 PM
Unknown Object (File)
Wed, Oct 2, 5:33 AM
Unknown Object (File)
Sep 6 2024, 2:36 PM
Unknown Object (File)
Sep 4 2024, 7:48 PM
Unknown Object (File)
Sep 1 2024, 9:28 PM
Unknown Object (File)
Aug 28 2024, 9:48 PM
Unknown Object (File)
Aug 28 2024, 7:59 AM
Subscribers
None

Details

Summary

Ref T10077. Currently, we issue 6+ queries on every page to build this menu, since the menu is built application-by-application.

Build the menu with dedicated modules instead so a single "EditEngine" module can provide all of them with one query.

I'd like to reduce this to 0 queries but I'm not totally sure what we want to do with this menu.

This change removes these items, because EditEngine can not currently provide them:

  • Calendar: Eventually via EditEngine eventually.
  • Conpherence: Probably via EditEngine, doesn't seem too important.
  • People: Maybe via EditEngine, doesn't seem too important? "Welcome" is likely better?
  • Pholio: Eventually via EditEngine.

It adds a bunch of other items as a side effect:

Screen Shot 2016-06-05 at 7.26.04 AM.png (417×220 px, 30 KB)

This reduces the queries issued on every page by ~5.

This also makes quick create actions visible while logged out (see T7073).

Test Plan
  • Viewed menu while logged in.
  • Viewed menu while logged out.
  • Viewed standalone version of menu.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Fully modularize the "Quick Actions" menu.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.

I'm not sure what the long-term version of this menu looks like. This moves slightly toward maybe putting some things like flags into it.

(Personally, I never use this menu.)

The shortest-effort, reasonable-ish path is probably to add an option to let applications opt out of the menu, and then hide badges, blogs/posts, EditEngine forms, and packages (leaving tasks, pastes and projects -- do we even need projects?).

I can force other stuff (people, pholio, etc) back into the menu for now if you don't want to remove them.

chad edited edge metadata.

I'll file something

This revision is now accepted and ready to land.Jun 5 2016, 5:18 PM
This revision was automatically updated to reflect the committed changes.