Page MenuHomePhabricator
Differential D15377

Give PHUITwoColumnView an addPropertySection method
ClosedPublic
Actions

Authored by chad on Mar 1 2016, 8:16 PM.
Tags
None
Referenced Files
F15520732: D15377.id37087.diff
Sun, Apr 20, 7:46 AM
F15476608: D15377.diff
Mon, Apr 7, 7:50 AM
F15428705: D15377.diff
Mar 23 2025, 9:48 PM
F15411379: D15377.diff
Mar 19 2025, 9:16 AM
F15398296: D15377.id37072.diff
Mar 16 2025, 11:59 PM
F15397814: D15377.id37087.diff
Mar 16 2025, 10:24 PM
F15394652: D15377.id37087.diff
Mar 15 2025, 11:59 PM
F15392749: D15377.diff
Mar 15 2025, 5:28 PM
View All Files
Subscribers
Korvin

Details

Reviewers
epriestley
Commits
Restricted Diffusion Commit
rPcaadd1025a1f: Give PHUITwoColumnView an addPropertySection method
Summary

Simplifies building pages a little more, adds a helper method to just add a property section to the main column automatically above other content.

Test Plan

Review Ponder, Herald, Passphrase, Countdown.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

chad updated this revision to Diff 37072.Mar 1 2016, 8:16 PM
chad retitled this revision from to Give PHUITwoColumnView an addPropertySection method.
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 TranscriptMar 1 2016, 8:16 PM
chad added a comment.Mar 2 2016, 5:04 PM

Mind taking a quick peek here? I'd like to simplify some of these calls.

epriestley accepted this revision.Mar 2 2016, 5:09 PM
epriestley edited edge metadata.

Seems fine to me.

This revision is now accepted and ready to land.Mar 2 2016, 5:09 PM
Closed by commit rPcaadd1025a1f: Give PHUITwoColumnView an addPropertySection method (authored by chad, committed by chad). · Explain WhyMar 2 2016, 5:35 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
src/
applications/
herald/
controller/
15 lines
passphrase/
controller/
12 lines
ponder/
controller/
13 lines
view/
phui/
39 lines

Diff 37087

View Options

src/applications/herald/controller/HeraldRuleViewController.php

Loading...
View Options

src/applications/passphrase/controller/PassphraseCredentialViewController.php

Loading...
View Options

src/applications/ponder/controller/PonderQuestionViewController.php

Loading...
View Options

src/view/phui/PHUITwoColumnView.php

Loading...
Phabricator · Built by Phacility