Page MenuHomePhabricator

Clean up AphrontError boxes, Diffusion Headers
ClosedPublic

Authored by chad on Jan 9 2014, 4:24 AM.
Tags
None
Referenced Files
F18932333: D7914.id.diff
Mon, Nov 10, 7:08 AM
F18909089: D7914.diff
Sun, Nov 9, 1:43 AM
F18806591: D7914.diff
Sat, Oct 18, 8:25 PM
F18777220: D7914.id17912.diff
Oct 11 2025, 1:22 AM
F18656797: D7914.diff
Sep 22 2025, 11:28 PM
F18647653: D7914.diff
Sep 19 2025, 2:03 PM
F18599322: D7914.id.diff
Sep 13 2025, 7:11 AM
F18588831: D7914.diff
Sep 12 2025, 1:40 AM

Details

Summary

Two basic changes here, first we fixed up the Diffusion headers to roll out more PHUIObjectBoxes. Second we added some specific styles for when Errors are inside an ObjectBox at the first position.

Test Plan

Tested a number of different layouts for browsing respositories as well as wherever I could find cases with PHUIObjectBox Form Errors (see images attached). Still some minor tightening due after this diff, but didnt want to overload it.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

{F100141}

{F100142}

{F100143}

{F100144}

{F100145}

Also, I seem to be stuck at importing.. haha.. need to troubleshoot that.