Page MenuHomePhabricator

Make inline reticle code more general and robust
ClosedPublic

Authored by epriestley on Mar 7 2015, 7:16 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Apr 25, 1:49 AM
Unknown Object (File)
Mar 1 2024, 2:19 PM
Unknown Object (File)
Mar 1 2024, 11:52 AM
Unknown Object (File)
Feb 20 2024, 4:31 AM
Unknown Object (File)
Feb 14 2024, 10:26 PM
Unknown Object (File)
Feb 14 2024, 10:26 PM
Unknown Object (File)
Feb 14 2024, 7:30 PM
Unknown Object (File)
Feb 11 2024, 9:38 AM
Subscribers

Details

Summary

Ref T2009. Currently, the code which draws the reticle is sort of implicitly hard-coded with some of the rules for the 2up view.

Instead, use general rules:

  • Start selection at the next <td />.
  • End selection at the rightmost adjacent <td />.

These rules work in all cases.

Test Plan
  • Activated reticle in 1up and 2up views by clicking line numbers and hovering over comments. It now draws correctly.
  • Dragged over line ranges in 1up and 2up views, saw accurate reticle.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Make inline reticle code more general and robust.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added reviewers: chad, btrahan.
chad edited edge metadata.
This revision is now accepted and ready to land.Mar 7 2015, 7:33 PM
This revision was automatically updated to reflect the committed changes.