Page MenuHomePhabricator
Differential D12495

Let ghostly inlines automatically expand changesets in enormous diffs
ClosedPublic
Actions

Authored by epriestley on Apr 21 2015, 7:57 PM.
Tags
None
Referenced Files
F18815690: D12495.diff
Tue, Oct 21, 3:13 AM
F18779551: D12495.id30006.diff
Sat, Oct 11, 2:26 PM
F18743106: D12495.id.diff
Thu, Oct 2, 10:24 PM
F18505452: D12495.id.diff
Sep 5 2025, 12:40 AM
F18497334: D12495.diff
Sep 4 2025, 5:59 PM
F18092532: D12495.id30006.diff
Aug 7 2025, 11:11 AM
F17929351: D12495.id.diff
Jul 30 2025, 5:01 PM
F17916896: D12495.diff
Jul 29 2025, 10:51 PM
View All Files
Subscribers
epriestley
yelirekim

Details

Reviewers
btrahan
Maniphest Tasks
T7447: Bring inline comments forward across revision updates
T7870: List of cases where ghost comments end up somewhere confusing or unexpected
Commits
Restricted Diffusion Commit
rPa4a65bd5f1d6: Let ghostly inlines automatically expand changesets in enormous diffs
Summary

Ref T7447. Ref T7870. When a diff affects more than 100 files, we collapse files by default, then expand files with inlines.

Ghostly inlines currently don't cause files to expand, but reasonably should.

Test Plan

Before:

expdn.png (713×1 px, 74 KB)

After:

Screen_Shot_2015-04-21_at_12.51.44_PM.png (1×1 px, 160 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Revision Contents
Changeset List

PathSize
src/
applications/
differential/
controller/
215 lines
39 lines
query/
188 lines

Diff 30006

View Options

src/applications/differential/controller/DifferentialChangesetViewController.php

Loading...
View Options

src/applications/differential/controller/DifferentialRevisionViewController.php

Loading...
View Options

src/applications/differential/query/DifferentialInlineCommentQuery.php

Loading...
Phabricator · Built by Phacility