Page MenuHomePhabricator

Clean up Profile Menu Item page
ClosedPublic

Authored by chad on Dec 12 2016, 4:27 AM.
Tags
None
Referenced Files
F19120115: D17030.id40966.diff
Dec 9 2025, 9:40 PM
F19065541: D17030.id40965.diff
Nov 30 2025, 2:36 AM
F19061161: D17030.id.diff
Nov 29 2025, 11:39 AM
F19060021: D17030.diff
Nov 29 2025, 7:38 AM
F19058056: D17030.id.diff
Nov 29 2025, 12:39 AM
F19039437: D17030.diff
Nov 26 2025, 7:46 AM
F18998778: D17030.id40967.diff
Nov 20 2025, 7:31 PM
F18970852: D17030.id40967.diff
Nov 15 2025, 9:11 AM
Subscribers

Details

Summary

Cleans up the UI on the page here, uses two column layout, places actions as actionlist instead of dropdown. Changes edit pages to dialogs.

Test Plan

Add an application, divider, link, and facts to a menu page.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

chad retitled this revision from to Clean up Profile Menu Item page.
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added a reviewer: epriestley.
epriestley edited edge metadata.

Nice, this looks a lot better. That dropdown was definitely a bit weird.

src/applications/search/engine/PhabricatorProfileMenuEngine.php
634

Do we need this?

This revision is now accepted and ready to land.Dec 12 2016, 4:04 PM
chad edited edge metadata.
  • comment
This revision was automatically updated to reflect the committed changes.