Page MenuHomePhabricator

Improve offset/range inline behavior for rich diffs and unified diffs
ClosedPublic

Authored by epriestley on May 14 2020, 10:53 PM.
Tags
None
Referenced Files
F15463526: D21257.id50618.diff
Wed, Apr 2, 12:30 AM
F15463316: D21257.id50619.diff
Tue, Apr 1, 10:24 PM
F15459771: D21257.id.diff
Mon, Mar 31, 4:10 PM
F15459424: D21257.diff
Mon, Mar 31, 12:46 PM
F15456267: D21257.diff
Sun, Mar 30, 8:21 AM
F15414826: D21257.diff
Thu, Mar 20, 2:40 AM
F15333100: D21257.id50618.diff
Fri, Mar 7, 11:51 PM
F15332283: D21257.id.diff
Fri, Mar 7, 5:00 PM
Subscribers
None

Details

Summary

Ref T13513. The way I'm highlighting lines won't work for Jupyter notebooks or other complex content blocks, and I don't see an obvious way to make it work that's reasonably robust.

However, we can just ignore the range behavior for complex content and treat the entire block as selected. This isn't quite as fancy as the source behavior, but pretty good.

Also, adjust unified diff behavior to work correctly with highlighting and range selection.

Test Plan
  • Used range selection in a Jupyter notebook, got reasonable behavior (range is treated as "entire block").
  • Used range selection in a unified diff, got equivalent behavior to 2-up diffs.

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.May 14 2020, 11:02 PM
This revision was automatically updated to reflect the committed changes.