Page MenuHomePhabricator

Add a footer to PHUITwoColumnView
ClosedPublic

Authored by chad on Mar 10 2016, 3:06 AM.
Tags
None
Referenced Files
F14005972: D15455.id37247.diff
Sun, Oct 27, 10:53 PM
F14005971: D15455.id37246.diff
Sun, Oct 27, 10:53 PM
F14004488: D15455.id.diff
Sat, Oct 26, 9:24 PM
F13999951: D15455.diff
Thu, Oct 24, 6:14 PM
F13981933: D15455.id.diff
Sat, Oct 19, 8:02 PM
F13965862: D15455.id37246.diff
Wed, Oct 16, 4:44 AM
F13964929: D15455.id37247.diff
Tue, Oct 15, 10:46 PM
Unknown Object (File)
Fri, Oct 11, 9:14 PM
Subscribers

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

I suppose I can remove ActionList and PropertyList

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

Go us!

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.

chad edited edge metadata.
  • remove lots of code

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

This revision was automatically updated to reflect the committed changes.

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