Page MenuHomePhabricator

Remove duplicate inline scaffold in 2up renderer
ClosedPublic

Authored by epriestley on Mar 6 2015, 2:04 PM.
Tags
None
Referenced Files
Unknown Object (File)
Apr 17 2026, 8:02 PM
Unknown Object (File)
Apr 15 2026, 7:55 AM
Unknown Object (File)
Apr 10 2026, 9:22 PM
Unknown Object (File)
Apr 4 2026, 11:00 PM
Unknown Object (File)
Mar 10 2026, 2:30 AM
Unknown Object (File)
Mar 7 2026, 7:39 PM
Unknown Object (File)
Mar 5 2026, 1:58 PM
Unknown Object (File)
Feb 26 2026, 11:31 AM
Subscribers

Details

Summary

Ref T2009. Remove the 4 (!!) copies of this code.

Test Plan
  • Added, edited, and removed inline comments in 2up view.
  • Stacked a bunch of comments on the same line and saw the JS place them correctly.
  • Created an image diff and added, edited and removed inlines on it.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Remove duplicate inline scaffold in 2up renderer.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
btrahan edited edge metadata.
This revision is now accepted and ready to land.Mar 6 2015, 10:59 PM
This revision was automatically updated to reflect the committed changes.