Page MenuHomePhabricator

Basic touchups to Phortune UI
ClosedPublic

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