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
Unknown Object (File)
Tue, Apr 16, 6:32 PM
Unknown Object (File)
Sun, Apr 14, 4:00 AM
Unknown Object (File)
Thu, Apr 11, 4:06 PM
Unknown Object (File)
Thu, Apr 11, 9:42 AM
Unknown Object (File)
Sun, Apr 7, 10:09 AM
Unknown Object (File)
Mon, Apr 1, 2:08 AM
Unknown Object (File)
Sun, Mar 24, 7:02 AM
Unknown Object (File)
Mar 12 2024, 2:21 AM
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