Page MenuHomePhabricator

Make taller tables the default for AphrontTableView
ClosedPublic

Authored by chad on Apr 8 2015, 3:24 PM.
Tags
None
Referenced Files
F18844542: D12328.id.diff
Wed, Oct 29, 6:17 AM
F18841008: D12328.diff
Tue, Oct 28, 5:49 AM
F18800658: D12328.id29618.diff
Fri, Oct 17, 4:36 PM
F18771339: D12328.diff
Oct 8 2025, 7:14 PM
F18220425: D12328.id29615.diff
Aug 19 2025, 2:44 PM
F18166021: D12328.id29618.diff
Aug 15 2025, 9:14 AM
F17819481: D12328.id.diff
Jul 26 2025, 1:15 AM
F17802519: D12328.id29618.diff
Jul 25 2025, 9:11 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.