HomePhabricator

Make taller tables the default for AphrontTableView

Description

Make taller tables the default for AphrontTableView

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.

Reviewers: btrahan, epriestley

Reviewed By: epriestley

Subscribers: Korvin, epriestley

Differential Revision: https://secure.phabricator.com/D12328

Details

Provenance
chadAuthored on
chadPushed on Apr 8 2015, 3:38 PM
Reviewer
epriestley
Differential Revision
D12328: Make taller tables the default for AphrontTableView
Parents
rP13c0c3b850ea: Fix "To: Unknown Object" on outbound Maniphest mail with no owner
Branches
Unknown
Tags
Unknown