Page MenuHomePhabricator

Update Phortune for two column, spruce up UI
ClosedPublic

Authored by chad on Mar 23 2016, 5:33 PM.
Tags
None
Referenced Files
F18833772: D15513.diff
Sun, Oct 26, 5:41 AM
F18824595: D15513.diff
Thu, Oct 23, 4:39 PM
F18821248: D15513.id37400.diff
Wed, Oct 22, 9:06 PM
F18686546: D15513.diff
Fri, Sep 26, 12:56 PM
F18620017: D15513.diff
Sep 15 2025, 4:01 AM
F18601101: D15513.diff
Sep 13 2025, 11:31 AM
F18592811: D15513.diff
Sep 12 2025, 1:14 PM
Unknown Object (File)
Jun 1 2025, 1:15 AM
Subscribers

Details

Reviewers
epriestley
Commits
Restricted Diffusion Commit
rP881785aba4bc: Update Phortune for two column, spruce up UI
Summary

Moves everything I could find in Phortune to new UI layouts.

Test Plan

Tested every page I could get two, unclear how to test subscriptions.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

chad retitled this revision from to Update Phortune for two column, spruce up UI.
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added a reviewer: epriestley.
epriestley edited edge metadata.

Only caught minor stuff:

  • "Details" box on subscriptions is a little whack, I can fix this if you can't test it locally:

Screen Shot 2016-03-23 at 10.38.53 AM.png (279×1 px, 36 KB)

  • Cancel button URI on high security checkpoint for managing a subscription is wrong (probably pre-existing, I'll send you a diff).
  • Merchant member would be nice in the sidebar. :3
  • Merchant "Description" is not separate (inconsistent?).

Screen Shot 2016-03-23 at 10.41.58 AM.png (501×1 px, 53 KB)

src/applications/phortune/controller/PhortuneSubscriptionViewController.php
85

Should be setHeaderText() to fix header maybe?

This revision is now accepted and ready to land.Mar 23 2016, 5:45 PM
chad edited edge metadata.
  • updates per commentittoes
This revision was automatically updated to reflect the committed changes.