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

	F15484626: D18527.id44533.diff
	Wed, Apr 9, 7:16 PM

	F15482084: D18527.id44533.diff
	Wed, Apr 9, 2:47 AM

	F15482032: D18527.id44491.diff
	Wed, Apr 9, 2:14 AM

	F15481944: D18527.id44492.diff
	Wed, Apr 9, 1:03 AM

	F15473995: D18527.diff
	Sun, Apr 6, 3:50 AM

	F15466474: D18527.id44532.diff
	Thu, Apr 3, 5:57 AM

	F15466266: D18527.id.diff
	Thu, Apr 3, 3:36 AM

	F15465604: D18527.diff
	Wed, Apr 2, 9:11 PM

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

Branch

edit-engine-white

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 18315
Build 24648: Run Core Tests
Build 24647: arc lint + arc unit

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 44532

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...