Page MenuHomePhabricator

Basic touchups to Phortune UI
ClosedPublic

Authored by chad on Mar 20 2015, 6:52 PM.
Tags
None
Referenced Files
F17808830: D12115.id29262.diff
Fri, Jul 25, 4:19 PM
F17803595: D12115.id29138.diff
Fri, Jul 25, 10:21 AM
F17802885: D12115.id.diff
Fri, Jul 25, 9:36 AM
F17765397: D12115.diff
Wed, Jul 23, 1:52 AM
F17762507: D12115.id.diff
Tue, Jul 22, 10:10 PM
F17676759: D12115.diff
Sun, Jul 13, 10:14 AM
Unknown Object (File)
Jun 20 2025, 10:25 AM
Unknown Object (File)
Jun 18 2025, 8:31 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
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

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.
btrahan edited edge metadata.

These tweaks all seem reasonable to me. Thanks!

This revision is now accepted and ready to land.Mar 23 2015, 5:35 PM
This revision was automatically updated to reflect the committed changes.