Page MenuHomePhabricator

Mention the "inline comments" rule in the callout for "Large" diffs
ClosedPublic

Authored by epriestley on May 6 2018, 1:32 PM.
Tags
None
Referenced Files
F13247583: D19430.diff
Thu, May 23, 8:42 PM
F13247477: D19430.id.diff
Thu, May 23, 7:34 PM
F13245460: D19430.diff
Thu, May 23, 6:02 AM
F13216749: D19430.diff
Sat, May 18, 2:05 AM
F13200819: D19430.diff
Tue, May 14, 3:54 AM
Unknown Object (File)
Fri, May 3, 5:02 PM
Unknown Object (File)
Mon, Apr 29, 5:03 PM
Unknown Object (File)
Apr 25 2024, 5:16 PM
Subscribers
None

Details

Summary

See PHI638. When a diff is large (between 100 and 1000 files), we collapse content by default unless a change also has inline comments.

This rule isn't explicitly explained anywhere. Although it's not really a critical rule, it fits easily enough into the UI callout.

Also render the UI callout in a slightly more modern way and avoid hsprintf().

Test Plan

Screen Shot 2018-05-06 at 6.30.16 AM.png (1×1 px, 216 KB)

  • Also, clicked the "Expand" link and saw everything expand properly.

Diff Detail

Repository
rP Phabricator
Branch
diff1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 20287
Build 27540: Run Core Tests
Build 27539: arc lint + arc unit