Page MenuHomePhabricator
Differential D17213

Load global and custom profile menu items in a single query
ClosedPublic
Actions

Authored by epriestley on Jan 17 2017, 8:49 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Apr 8, 3:23 PM
Unknown Object (File)
Sat, Apr 6, 10:05 PM
Unknown Object (File)
Mon, Apr 1, 6:15 PM
Unknown Object (File)
Mar 15 2024, 3:02 PM
Unknown Object (File)
Mar 9 2024, 1:14 PM
Unknown Object (File)
Feb 14 2024, 10:37 AM
Unknown Object (File)
Jan 25 2024, 12:04 AM
Unknown Object (File)
Dec 27 2023, 9:28 AM
View All 93 Files
Subscribers
None

Details

Reviewers
chad
Maniphest Tasks
T5867: Move Quick Create to MenuItemEngine
Commits
rPd7e5a8b978ad: Load global and custom profile menu items in a single query
Summary

Ref T5867. Use a single query to load both personal and global items, then reorder them and add a divider if both groups have some stuff.

Test Plan

Viewed menu, edited personal and global items, viewed/edited existing project menus.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley created this revision.Jan 17 2017, 8:49 PM
Harbormaster completed remote builds in B15293: Diff 41404.Jan 17 2017, 8:50 PM
chad edited edge metadata.Jan 17 2017, 8:52 PM

oh yea, so easy

magicb41e5.gif (307×570 px, 7 MB)

chad accepted this revision.Jan 17 2017, 8:54 PM
This revision is now accepted and ready to land.Jan 17 2017, 8:54 PM
epriestley added a comment.Jan 17 2017, 9:02 PM

The builtin stuff made this a bit more involved than I imagined, yeah.

Closed by commit rPd7e5a8b978ad: Load global and custom profile menu items in a single query (authored by epriestley, committed by epriestley). · Explain WhyJan 17 2017, 9:02 PM
This revision was automatically updated to reflect the committed changes.
epriestley added a comment.Jan 17 2017, 9:04 PM

Also, any clue where this came from?

Screen Shot 2017-01-17 at 1.04.32 PM.png (164×148 px, 10 KB)

chad added a comment.Jan 17 2017, 9:17 PM

Hmm I didn't see it in my testing but probably still me. Will look soon (getting MacBook fixed)

epriestley added a comment.Jan 17 2017, 9:35 PM

Also, why is the label in the user menu the Real Name, but log out still says "Log out <username>"?

chad added a comment.Jan 17 2017, 10:01 PM

I haven't built the final version yet. This diff was getting long in the tooth. I think I commented above? I have more plans for UI.

epriestley added a comment.Jan 17 2017, 10:01 PM

Ah, must have missed it.

chad added a comment.Jan 17 2017, 10:05 PM

I think I need to convert PhabricatorActionView to a proper AphrontTagView, and migrate setMetadata globally to be able to fully customize the UI there. Brings us closer to PHUIListItemView as well.

Revision Contents
Changeset List

Diff 41405

View Options

src/applications/favorites/application/PhabricatorFavoritesApplication.php

Loading...
View Options

src/applications/favorites/engine/PhabricatorFavoritesProfileMenuEngine.php

Loading...
View Options

src/applications/search/engine/PhabricatorProfileMenuEngine.php

Loading...
View Options

src/applications/search/query/PhabricatorProfileMenuItemConfigurationQuery.php

Loading...
View Options

src/applications/search/storage/PhabricatorProfileMenuItemConfiguration.php

Loading...
Phabricator · Built by Phacility