Page MenuHomePhabricator

Never render "Show More Context" inside an inline comment suggestion diff
ClosedPublic

Authored by epriestley on Oct 2 2020, 4:44 PM.
Tags
None
Referenced Files
F18109136: D21476.id51118.diff
Mon, Aug 11, 11:50 AM
F18104359: D21476.id51119.diff
Sun, Aug 10, 11:43 AM
F18101398: D21476.id.diff
Sat, Aug 9, 12:51 PM
F18096501: D21476.diff
Fri, Aug 8, 7:19 AM
F17757563: D21476.diff
Jul 22 2025, 4:15 PM
F17756209: D21476.id51119.diff
Jul 22 2025, 1:51 PM
F17756208: D21476.id51118.diff
Jul 22 2025, 1:51 PM
F17756015: D21476.id.diff
Jul 22 2025, 1:32 PM
Subscribers
None

Details

Summary

See PHI1896. If you do this:

  • Create an inline comment over a wide range of lines.
  • Suggest an edit.
  • Make a change near the beginning of the block.
  • Make a change near the end of the block.
  • Save the inline.

...you get a rendering which includes a "Show More Context" fold in the middle.

Currently, this element renders in a visually broken way and consumes too many columns.

However, this element isn't ever desirable inside inline comment suggestions. Stop it from rendering entirely.

Test Plan
  • Made an inline comment suggestion across lines 1-50 with edits at the beginning and end, saw a contiguous diff.
  • Made smaller inline comment suggestions (one line, a few lines).

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

This revision was not accepted when it landed; it landed in state Needs Review.Oct 2 2020, 4:46 PM
epriestley requested review of this revision.
This revision was automatically updated to reflect the committed changes.