Page MenuHomePhabricator

Make line selection in source code views less fragile and more consistent
ClosedPublic

Authored by epriestley on Apr 11 2018, 9:15 PM.
Tags
None
Referenced Files
Unknown Object (File)
Mon, Jun 2, 1:33 PM
Unknown Object (File)
Wed, May 28, 10:23 PM
Unknown Object (File)
Tue, May 27, 10:56 PM
Unknown Object (File)
Apr 30 2025, 10:05 AM
Unknown Object (File)
Apr 25 2025, 10:19 PM
Unknown Object (File)
Apr 24 2025, 9:57 PM
Unknown Object (File)
Apr 9 2025, 8:43 AM
Unknown Object (File)
Apr 8 2025, 9:51 AM
Subscribers
None

Details

Summary

Depends on D19347. Ref T13105. See PHI565. The "highlight lines" behavior is interacting poorly with the new blame element in Diffusion.

Make the behavior a little simpler and hopefully more robust.

Test Plan
  • Clicked commit/revision links in Diffusion, saw the links get followed instead of the lines highlighted.
  • Highlighted lines in Diffusion, saw just the line/code highlight instead of the whole thing.
  • Highlighted lines in Paste and new-style Harbormaster build logs, saw consistent behavior.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable