Clean up Profile Menu Item page
ClosedPublic
Actions

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

Tags

None

Referenced Files

	F13237971: D17030.id40966.diff
	Tue, May 21, 4:18 PM

	F13215798: D17030.id40967.diff
	Fri, May 17, 7:52 PM

	F13204370: D17030.diff
	Wed, May 15, 12:31 AM

	F13187287: D17030.diff
	Sat, May 11, 4:26 AM

	Unknown Object (File)
	Tue, May 7, 7:33 AM

	Unknown Object (File)
	Fri, May 3, 6:52 AM

	Unknown Object (File)
	Tue, Apr 30, 1:09 AM

	Unknown Object (File)
	Thu, Apr 25, 1:18 AM

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