Page MenuHomePhabricator
Differential D15455

Add a footer to PHUITwoColumnView
ClosedPublic
Actions

Authored by chad on Mar 10 2016, 3:06 AM.
Tags
None
Referenced Files
F18045076: D15455.id.diff
Sun, Aug 3, 8:21 AM
F17997260: D15455.diff
Sat, Aug 2, 4:53 AM
F17830515: D15455.id37248.diff
Sat, Jul 26, 11:07 AM
F17623537: D15455.id37248.diff
Thu, Jul 10, 1:02 AM
F17623249: D15455.id37248.diff
Thu, Jul 10, 12:34 AM
Unknown Object (File)
Jun 27 2025, 3:38 AM
Unknown Object (File)
Jun 26 2025, 8:16 PM
Unknown Object (File)
Jun 26 2025, 5:21 PM
View All Files
Subscribers
Korvin

Details

Reviewers
epriestley
Commits
Restricted Diffusion Commit
rPe351eba74469: Add a footer to PHUITwoColumnView
Summary

This allows setting of full-width content underneath the two column, or full column all by itself. Maybe these names are bad.

Test Plan

Using these in Differential / Diffusion conversions.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

chad updated this revision to Diff 37246.Mar 10 2016, 3:06 AM
chad retitled this revision from to Add a footer to PHUITwoColumnView.
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 10 2016, 3:06 AM
epriestley accepted this revision.Mar 10 2016, 3:09 AM
epriestley edited edge metadata.

Is this any different than this?

return $this->newPage()
  ->setBlahBlah(...)
  ->appendChild(
    array(
      $two_column_view,
      $some_more_content,
    ));
This revision is now accepted and ready to land.Mar 10 2016, 3:09 AM
chad added a comment.Mar 10 2016, 3:09 AM

I suppose I can remove ActionList and PropertyList

epriestley added a comment.Mar 10 2016, 3:10 AM

Oh yeah, we probably got rid of all of those.

Go us!

chad added a comment.Mar 10 2016, 3:10 AM

Yes, it has built in page margins. I want to get away from setting margins for each component for different viewports and just use a page layout engine.

epriestley added a comment.Mar 10 2016, 3:10 AM

Ah, makes sense.

chad updated this revision to Diff 37247.Mar 10 2016, 3:16 AM
chad edited edge metadata.
  • remove lots of code
chad added a comment.Mar 10 2016, 3:17 AM

I suppose even this might move to subclassing newPage ? newObjectPage ?

Closed by commit rPe351eba74469: Add a footer to PHUITwoColumnView (authored by chad, committed by chad). · Explain WhyMar 10 2016, 3:18 AM
This revision was automatically updated to reflect the committed changes.
chad added a comment.Mar 10 2016, 3:21 AM

I am trying to think what would make something like a redesign take less work in the future.

Revision Contents
Changeset List

PathSize
src/
view/
phui/
54 lines

Diff 37248

View Options

src/view/phui/PHUITwoColumnView.php

Loading...
Phabricator · Built by Phacility