Clean up Profile Menu Item page
ClosedPublic
Actions

Authored by chad on Dec 12 2016, 4:27 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Fri, Dec 13, 6:23 PM

	Unknown Object (File)
	Thu, Dec 12, 10:24 PM

	Unknown Object (File)
	Wed, Dec 11, 11:01 AM

	Unknown Object (File)
	Sun, Dec 8, 4:53 PM

	Unknown Object (File)
	Wed, Nov 27, 12:35 AM

	Unknown Object (File)
	Tue, Nov 26, 9:08 PM

	Unknown Object (File)
	Fri, Nov 22, 6:18 PM

	Unknown Object (File)
	Nov 17 2024, 7:34 PM

View All Files

Subscribers

Korvin

Details

Reviewers

epriestley

Commits

rPd8b028b51bc0: Clean up Profile Menu Item page

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 updated this revision to Diff 40965.Dec 12 2016, 4:27 AM

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.

Herald added a subscriber: Korvin. · View Herald TranscriptDec 12 2016, 4:27 AM

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

comment

Closed by commit rPd8b028b51bc0: Clean up Profile Menu Item page (authored by chad, committed by chad). · Explain WhyDec 12 2016, 4:38 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

search/

engine/

PhabricatorProfileMenuEngine.php

49 lines

Diff 40967

View Options

src/applications/search/engine/PhabricatorProfileMenuEngine.php

Clean up Profile Menu Item pageClosedPublicActions