HomePhorge

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
Chad LittleAuthored on Apr 8 2015, 8:38 AM
themackabuPushed on Mar 25 2025, 8:07 PM
Parents
rP13c0c3b850ea: Fix "To: Unknown Object" on outbound Maniphest mail with no owner
Branches
Unknown
Tags
Unknown

Event Timeline