Paths

Table of Contentst

Differential D12115

Basic touchups to Phortune UI
ClosedPublic
Actions

Authored by chad on Mar 20 2015, 6:52 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Wed, Nov 20, 9:35 AM

	Unknown Object (File)
	Sat, Nov 16, 6:44 AM

	Unknown Object (File)
	Tue, Nov 12, 11:18 AM

	Unknown Object (File)
	Tue, Nov 12, 4:28 AM

	Unknown Object (File)
	Sun, Nov 10, 10:22 AM

	Unknown Object (File)
	Fri, Nov 8, 11:17 AM

	Unknown Object (File)
	Mon, Nov 4, 10:47 PM

	Unknown Object (File)
	Oct 22 2024, 1:38 AM

Subscribers

Details

Reviewers

epriestley
btrahan

Commits

Restricted Diffusion Commit
rPc4799b2c4535: Basic touchups to Phortune UI

Summary

This is just a quick pass to fix a few bugs and spacing issues, Phortune itself could probably use some more custom UI, but that'll require some thought and abstraction. This also adds a new taller table CSS, which I mayyyy make automatic on tables with few rows, we'll see.

Test Plan

Browsed my Phortune account, tested new spacing on admin for 'full effect'

Diff Detail

Repository

Branch

phortune

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 4928
Build 4946: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

chad updated this revision to Diff 29138.Mar 20 2015, 6:52 PM

chad retitled this revision from to Basic touchups to Phortune UI.

chad updated this object.

chad edited the test plan for this revision. (Show Details)

chad added reviewers: epriestley, btrahan.

Herald added subscribers: epriestley, Korvin. · View Herald TranscriptMar 20 2015, 6:52 PM

These tweaks all seem reasonable to me. Thanks!

This revision is now accepted and ready to land.Mar 23 2015, 5:35 PM

Closed by commit rPc4799b2c4535: Basic touchups to Phortune UI (authored by chad, committed by chad). · Explain WhyMar 26 2015, 8:16 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

resources/

celerity/

6 lines

src/

applications/

phortune/

controller/

PhortuneAccountListController.php

5 lines

PhortuneMerchantViewController.php

1 line

view/

PhortuneChargeTableView.php

1 line

PhortuneOrderTableView.php

1 line

PhortuneSubscriptionTableView.php

1 line

view/

control/

AphrontTableView.php

22 lines

webroot/

rsrc/

css/

aphront/

9 lines

Diff 29138

resources/celerity/map.php

Loading...

src/applications/phortune/controller/PhortuneAccountListController.php

Loading...

src/applications/phortune/controller/PhortuneMerchantViewController.php

Loading...

src/applications/phortune/view/PhortuneChargeTableView.php

Loading...

src/applications/phortune/view/PhortuneOrderTableView.php

Loading...

src/applications/phortune/view/PhortuneSubscriptionTableView.php

Loading...

src/view/control/AphrontTableView.php

Loading...

webroot/rsrc/css/aphront/table-view.css

Loading...