Page MenuHomePhabricator

Use normalsized font sizes in AphrontTable
ClosedPublic

Authored by chad on Jun 6 2017, 10:49 PM.
Tags
None
Referenced Files
F15426213: D18088.diff
Sun, Mar 23, 7:44 AM
F15423828: D18088.id43499.diff
Sat, Mar 22, 6:43 PM
F15419189: D18088.diff
Fri, Mar 21, 4:17 AM
F15382826: D18088.id43499.diff
Fri, Mar 14, 2:36 PM
Unknown Object (File)
Mon, Mar 3, 3:00 PM
Unknown Object (File)
Jan 21 2025, 9:43 AM
Unknown Object (File)
Jan 20 2025, 7:50 PM
Unknown Object (File)
Jan 17 2025, 5:24 PM
Subscribers

Details

Summary

Updates to use the standard 13px size we use everywhere else, cleans up a few mobile/display bugs, adds a hover state for tr.

Test Plan

Review Diffusion, Daemons, Almanac, People Logs, anything else?

Screen Shot 2017-06-06 at 3.50.23 PM.png (2×3 px, 900 KB)

Screen Shot 2017-06-06 at 3.50.12 PM.png (2×3 px, 929 KB)

Diff Detail

Repository
rP Phabricator
Branch
table-spacing (branched from master)
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 17411
Build 23343: Run Core Tests
Build 23342: arc lint + arc unit

Event Timeline

This revision is now accepted and ready to land.Jun 6 2017, 11:16 PM

if the hover feels weird anywhere, lmk. I can add a noHover if needed, but it felt good generally in any table.

This revision was automatically updated to reflect the committed changes.