Page MenuHomePhabricator

Make taller tables the default for AphrontTableView
ClosedPublic

Authored by chad on Apr 8 2015, 3:24 PM.
Tags
None
Referenced Files
F19263907: D12328.id29615.diff
Mon, Dec 22, 6:34 PM
F18970071: D12328.id29615.diff
Nov 15 2025, 7:23 AM
F18844542: D12328.id.diff
Oct 29 2025, 6:17 AM
F18841008: D12328.diff
Oct 28 2025, 5:49 AM
F18800658: D12328.id29618.diff
Oct 17 2025, 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
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.