Page MenuHomePhabricator

Make taller tables the default for AphrontTableView
ClosedPublic

Authored by chad on Apr 8 2015, 3:24 PM.
Tags
None
Referenced Files
F19898981: D12328.id29618.diff
Sun, Mar 29, 5:02 PM
F19897128: D12328.id.diff
Sun, Mar 29, 6:34 AM
F19897126: D12328.id.diff
Sun, Mar 29, 6:34 AM
F19896488: D12328.id29615.diff
Sun, Mar 29, 3:47 AM
F19815559: D12328.diff
Mar 4 2026, 10:55 PM
F19789848: D12328.id29615.diff
Feb 25 2026, 6:42 PM
F19780340: D12328.id.diff
Feb 23 2026, 7:17 PM
F19286272: D12328.id29618.diff
Dec 23 2025, 6:22 AM
Subscribers

Details

Summary

I considered at the time just making all tables taller. This removes the special casing and adds the space universally. On first glance all smaller tables look great, but Diffusion seems a little bloated. After a short time period though that went away for me. I do think Diffusion overall needs a UI refresh.

Test Plan

Tested numerous tables in Phortune, Diffusion, etc. Spacing feels more readable.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

chad retitled this revision from to Make taller tables the default for AphrontTableView.
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added reviewers: epriestley, btrahan.
epriestley edited edge metadata.
This revision is now accepted and ready to land.Apr 8 2015, 3:29 PM
This revision was automatically updated to reflect the committed changes.