Page MenuHomePhabricator

Basic touchups to Phortune UI
ClosedPublic

Authored by chad on Mar 20 2015, 6:52 PM.
Tags
None
Referenced Files
F13818394: D12115.diff
Fri, Sep 20, 7:31 AM
Unknown Object (File)
Wed, Aug 28, 4:57 AM
Unknown Object (File)
Thu, Aug 22, 4:54 PM
Unknown Object (File)
Aug 17 2024, 5:02 PM
Unknown Object (File)
Aug 17 2024, 5:35 AM
Unknown Object (File)
Aug 17 2024, 1:29 AM
Unknown Object (File)
Jul 27 2024, 12:47 AM
Unknown Object (File)
Jul 22 2024, 1:03 PM
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.