Differential D19310

Make DocumentEngine source line linking behavior better when blame is shown
ClosedPublic
Actions

Authored by epriestley on Apr 9 2018, 12:16 PM.

Tags

None

Referenced Files

	F15458859: D19310.id46213.diff
	Mon, Mar 31, 7:55 AM

	F15455709: D19310.diff
	Sun, Mar 30, 4:23 AM

	F15455217: D19310.id46208.diff
	Sat, Mar 29, 10:56 PM

	F15450741: D19310.id.diff
	Fri, Mar 28, 5:53 PM

	F15448832: D19310.id.diff
	Fri, Mar 28, 7:06 AM

	F15447736: D19310.diff
	Fri, Mar 28, 1:19 AM

	F15440687: D19310.diff
	Wed, Mar 26, 2:07 PM

	F15432294: D19310.id46213.diff
	Mon, Mar 24, 5:20 PM

View All Files

Subscribers

None

Details

Reviewers: None
Maniphest Tasks: T13105: Plans: Rich presentation and diff rendering pipelines for various file types
Commits: rP11664277b33d: Make DocumentEngine source line linking behavior better when blame is shown

Summary

Ref T13105. The line linker behavior currently has trouble identifying the line number when blame is active. Improve this, albeit not the most cleanly.

Test Plan

Selected lines with blame on.

Diff Detail

Repository

rP Phabricator

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.Apr 9 2018, 12:16 PM

Harbormaster completed remote builds in B20032: Diff 46208.Apr 9 2018, 12:17 PM

epriestley requested review of this revision.Apr 9 2018, 12:17 PM

epriestley added a child revision: D19311: Use javelin_tag(), not phutil_tag(), to render revision blame tooltips properly.Apr 9 2018, 12:20 PM

This revision was not accepted when it landed; it landed in state Needs Review.Apr 9 2018, 1:09 PM

Closed by commit rP11664277b33d: Make DocumentEngine source line linking behavior better when blame is shown (authored by epriestley). · Explain Why

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

resources/

celerity/

map.php

16 lines

webroot/

rsrc/

js/

core/

behavior-line-linker.js

17 lines

Diff 46213

View Options

resources/celerity/map.php