Differential D18527

Update EditEngine pages to take a page header separate
ClosedPublic
Actions

Authored by chad on Sep 4 2017, 10:33 PM.

Tags

None

Referenced Files

	F18108523: D18527.id44532.diff
	Mon, Aug 11, 7:23 AM

	F18107918: D18527.id.diff
	Mon, Aug 11, 3:50 AM

	F18106461: D18527.diff
	Sun, Aug 10, 10:59 PM

	F18090957: D18527.diff
	Wed, Aug 6, 6:16 PM

	F18048744: D18527.id44492.diff
	Sun, Aug 3, 1:59 PM

	F17678652: D18527.id44491.diff
	Jul 13 2025, 1:32 PM

	F17677031: D18527.id44492.diff
	Jul 13 2025, 11:07 AM

	F17652444: D18527.id44532.diff
	Jul 12 2025, 6:44 AM

Subscribers

Details

Reviewers

Commits

rPa903388d4f89: Update EditEngine pages to take a page header separate

Summary

This simplifies EditEngine pages in general by removing the dual header, and extending to allow setting of a custom PHUIHeaderView if needed (like settings).

Test Plan

Review all settings pages, review task, project pages. This should all be fine, but is a big change maybe some layouts I'm not considering. Tested these all mobile, destkop as well.

image.png (1×2 px, 228 KB)

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

chad created this revision.Sep 4 2017, 10:33 PM

Herald added a subscriber: Korvin. · View Herald TranscriptSep 4 2017, 10:33 PM

Harbormaster completed remote builds in B18275: Diff 44491.Sep 4 2017, 10:34 PM

chad edited the test plan for this revision. (Show Details)Sep 4 2017, 10:35 PM

chad added a parent revision: D18526: Update Settings for WHITE_CONFIG style boxes.

remove unused var

Harbormaster completed remote builds in B18276: Diff 44492.Sep 4 2017, 11:15 PM

Harbormaster completed remote builds in B18276: Diff 44492.

chad added a child revision: D18528: Update Dashboards for new Edit UI.Sep 4 2017, 11:40 PM

chad added a child revision: D18529: Update Create Diff page for new Edit UI.Sep 4 2017, 11:45 PM

chad added a child revision: D18530: Update Pholio Edit pages to new UI.Sep 4 2017, 11:52 PM

chad added a child revision: D18531: Update Phriction with new Edit UI.Sep 4 2017, 11:59 PM

chad added a child revision: D18532: Update Drydock to new Edit UI.Sep 5 2017, 1:29 AM

epriestley accepted this revision.Sep 5 2017, 1:13 PM

This revision is now accepted and ready to land.Sep 5 2017, 1:13 PM

rebase

Closed by commit rPa903388d4f89: Update EditEngine pages to take a page header separate (authored by Chad Little <chad@phacility.com>). · Explain WhySep 6 2017, 3:07 AM

This revision was automatically updated to reflect the committed changes.

Harbormaster completed remote builds in B18315: Diff 44532.Sep 6 2017, 3:07 AM

Revision Contents
Changeset List

Path

Size

resources/

celerity/

6 lines

src/

applications/

settings/

editor/

PhabricatorSettingsEditEngine.php

25 lines

transactions/

editengine/

PhabricatorEditEngine.php

28 lines

view/

phui/

PHUIInfoView.php

1 line

PHUITwoColumnView.php

4 lines

webroot/

rsrc/

css/

phui/

phui-two-column-view.css

4 lines

Diff 44533

resources/celerity/map.php

Loading...

src/applications/settings/editor/PhabricatorSettingsEditEngine.php

Loading...

src/applications/transactions/editengine/PhabricatorEditEngine.php

Loading...

src/view/phui/PHUIInfoView.php

Loading...

src/view/phui/PHUITwoColumnView.php

Loading...

webroot/rsrc/css/phui/phui-two-column-view.css

Loading...