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
F19057806: D15377.diff
Fri, Nov 28, 11:47 PM
F18956721: D15377.id37072.diff
Wed, Nov 12, 10:56 PM
F18840313: D15377.id37087.diff
Oct 27 2025, 11:28 PM
F18756183: D15377.id.diff
Oct 5 2025, 9:55 AM
F18656693: D15377.id.diff
Sep 22 2025, 11:12 PM
F18600696: D15377.diff
Sep 13 2025, 10:20 AM
F18591273: D15377.diff
Sep 12 2025, 8:45 AM
F18586306: D15377.diff
Sep 11 2025, 6:12 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