Differential D12328

Make taller tables the default for AphrontTableView
ClosedPublic
Actions

Authored by chad on Apr 8 2015, 3:24 PM.

Tags

None

Referenced Files

	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

	F17819481: D12328.id.diff
	Jul 26 2025, 1:15 AM

Subscribers

Details

Reviewers

epriestley
btrahan

Commits

Restricted Diffusion Commit
rP21f0ce736269: 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.

Diff Detail

Repository

Branch

taller-tables

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 5203
Build 5221: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

chad updated this revision to Diff 29615.Apr 8 2015, 3:24 PM

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.

Herald added subscribers: epriestley, Korvin. · View Herald TranscriptApr 8 2015, 3:24 PM

epriestley accepted this revision.Apr 8 2015, 3:29 PM

epriestley edited edge metadata.

This revision is now accepted and ready to land.Apr 8 2015, 3:29 PM

Closed by commit rP21f0ce736269: Make taller tables the default for AphrontTableView (authored by chad, committed by chad). · Explain WhyApr 8 2015, 3:38 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

resources/

celerity/

22 lines

src/

applications/

phortune/

view/

PhortuneChargeTableView.php

1 line

PhortuneOrderTableView.php

1 line

PhortuneSubscriptionTableView.php

1 line

view/

control/

AphrontTableView.php

9 lines

webroot/

rsrc/

css/

aphront/

12 lines

js/

application/

diffusion/

behavior-commit-graph.js

2 lines

Diff 29615

resources/celerity/map.php

Loading...

src/applications/phortune/view/PhortuneChargeTableView.php

Loading...

src/applications/phortune/view/PhortuneOrderTableView.php

Loading...

src/applications/phortune/view/PhortuneSubscriptionTableView.php

Loading...

src/view/control/AphrontTableView.php

Loading...

webroot/rsrc/css/aphront/table-view.css

Loading...

webroot/rsrc/js/application/diffusion/behavior-commit-graph.js

Loading...