Page MenuHomePhabricator

Basic touchups to Phortune UI
ClosedPublic

Authored by chad on Mar 20 2015, 6:52 PM.
Tags
None
Referenced Files
F15407698: D12115.diff
Tue, Mar 18, 7:02 PM
F15397522: D12115.id.diff
Sun, Mar 16, 8:47 PM
F15380409: D12115.id29262.diff
Fri, Mar 14, 2:53 AM
F15310953: D12115.diff
Thu, Mar 6, 9:38 AM
F15302641: D12115.id29262.diff
Wed, Mar 5, 7:11 PM
Unknown Object (File)
Thu, Feb 27, 7:31 PM
Unknown Object (File)
Feb 17 2025, 4:23 PM
Unknown Object (File)
Feb 9 2025, 6:11 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.