" IDs on left-hand-side of image changesets
ClosedPublic
Actions

Authored by epriestley on May 17 2017, 12:54 PM.

Tags

None

Referenced Files

	F19543915: D17926.id43115.diff
	Fri, Jan 23, 1:32 AM

	F19305137: D17926.id43116.diff
	Dec 24 2025, 4:29 PM

	F18915748: D17926.id43116.diff
	Nov 9 2025, 8:00 AM

	F18906296: D17926.id.diff
	Nov 8 2025, 5:48 PM

	F18850207: D17926.diff
	Oct 30 2025, 3:04 PM

	F18819200: D17926.diff
	Oct 22 2025, 4:57 AM

	F18760388: D17926.id.diff
	Oct 6 2025, 9:08 AM

	F18757641: D17926.diff
	Oct 5 2025, 6:30 PM

View All Files

Subscribers

None

Details

Reviewers

chad

Maniphest Tasks

T7682: JS Errors when mousing over image diff comments, right side only

Commits

rP422eb9db837d: Correct the generation of "<th />" IDs on left-hand-side of image changesets

Summary

Fixes T7682. The left-hand-side "<th />" row did not generate with the correct ID.

(I couldn't reproduce the exact issue described in T7682, but hovering comments on either side now works properly for me.)

Test Plan

Screen Shot 2017-05-17 at 5.19.15 AM.png (1×2 px, 1 MB)

Diff Detail

Repository

rP Phabricator

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley created this revision.May 17 2017, 12:54 PM

Harbormaster completed remote builds in B17043: Diff 43115.May 17 2017, 12:55 PM

chad accepted this revision.May 17 2017, 1:22 PM

This revision is now accepted and ready to land.May 17 2017, 1:22 PM

Closed by commit rP422eb9db837d: Correct the generation of "<th />" IDs on left-hand-side of image changesets (authored by epriestley, committed by epriestley). · Explain WhyMay 17 2017, 1:26 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

differential/

render/

DifferentialChangesetTwoUpRenderer.php

6 lines

Diff 43116

View Options

src/applications/differential/render/DifferentialChangesetTwoUpRenderer.php

Correct the generation of "<th />" IDs on left-hand-side of image changesetsClosedPublicActions